Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update design tokens #316

Merged

Conversation

tuentisre
Copy link
Collaborator

This PR was automatically created by the import-design-tokens GitHub Action.

@tuentisre tuentisre force-pushed the import-design-tokens branch from 4128c11 to 8575ed6 Compare November 7, 2023 15:09
@tuentisre tuentisre requested review from a team, aweell, yamal-alm and pmartinbTEF and removed request for a team November 7, 2023 15:09
@dagonco dagonco mentioned this pull request Nov 7, 2023
6 tasks
Copy link
Contributor

@dpastor dpastor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@dagonco dagonco removed the request for review from pmartinbTEF November 7, 2023 15:13
Copy link

github-actions bot commented Nov 7, 2023

📱 New catalog for testing generated: Download

@@ -58,6 +58,117 @@
<item name="colorTextLinkSnackbar">@color/vivoNew_color_vivoPurpleLight50</item>
<item name="colorControl">@color/vivoNew_color_darkModeGrey6</item>
<item name="colorControlActive">@color/vivoNew_color_vivoPurpleLight80</item>
<item name="colorControlInverse">@color/vivoNew_color_darkModeGrey6</item>
Copy link
Member

@dagonco dagonco Nov 7, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aweell don't panic here!
Context: We have had to modify the script that generates the Mística styles due to a problem keeping Vivo and VivoNovoApp in the same brand. Android Issues. VivoNovoApp will be duplicated in this file. That's why in this file are lots of new colors.

image

@yceballost yceballost requested review from yceballost and removed request for aweell November 7, 2023 15:31
@dagonco dagonco merged commit 20bf70e into dagonco/ANDROID-13833-novo-app-override Nov 7, 2023
4 of 5 checks passed
@dagonco dagonco deleted the import-design-tokens branch November 7, 2023 15:53
dagonco added a commit that referenced this pull request Nov 7, 2023
* ANDROID-13833 Created a clone of VivoNew to mitigate a inheritance styling problem.

* ANDROID-13833 Omit compose customization

* ANDROID-13833 Add a new parameter in brand to handle inheritances

* ANDROID-13833 Copy VivoNew customizations

* ANDROID-13833 Delete old vivoNewOverriden files

* ANDROID-13833 Extract common functionality and add Override suffix

* Update design tokens (#316)

Co-authored-by: dagonco <[email protected]>

---------

Co-authored-by: Flow <[email protected]>
Co-authored-by: dagonco <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants