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

Missing whitelabel themes in wizard #6804

Closed
1 task
murilopereirame opened this issue Mar 27, 2024 · 6 comments · Fixed by #6819
Closed
1 task

Missing whitelabel themes in wizard #6804

murilopereirame opened this issue Mar 27, 2024 · 6 comments · Fixed by #6819
Labels
android issues that concern the android client but not all clients dev bug unpublished bugs, found during our development/test cycle (excluded from release notes) state:done meets our definition of done
Milestone

Comments

@murilopereirame
Copy link
Contributor

murilopereirame commented Mar 27, 2024

On Android one of the whitelabel themes is missing and to show the first one, I had to trigger the wizard several times through the about dialog.

Steps to reproduce:
On Android:

  1. Add an account with whitelabel theme
  2. Trigger the wizard
  3. On theme step, see that there's no whitelabel theme

Screenshot_20240327_174005_Tuta-Test
Screenshot_20240327_174012_Tuta-Test

Originally posted by @murilopereirame in #6770 (comment)

Test Notes

  • Whitelabel themes are displayed during the onboard wizard
@murilopereirame murilopereirame added the dev bug unpublished bugs, found during our development/test cycle (excluded from release notes) label Mar 27, 2024
@murilopereirame murilopereirame added this to the 220.240327.0 milestone Mar 27, 2024
@murilopereirame murilopereirame added the android issues that concern the android client but not all clients label Mar 27, 2024
@rezbyte rezbyte self-assigned this Mar 28, 2024
@rezbyte
Copy link
Contributor

rezbyte commented Mar 28, 2024

I cannot seem to reproduce this. In the second screenshot, it seems that the custom colors are disabled.

@rezbyte rezbyte removed their assignment Mar 28, 2024
@charlag
Copy link
Contributor

charlag commented Mar 28, 2024

Also on the first screenshot there clearly is a whitelabel theme, I will try to repro it as well but if not then I'd say it's okay

@murilopereirame
Copy link
Contributor Author

I cannot seem to reproduce this. In the second screenshot, it seems that the custom colors are disabled.

Good catch, but the color was enabled on the web. Could be just a server delay?

Also on the first screenshot there clearly is a whitelabel theme, I will try to repro it as well but if not then I'd say it's okay

Yes, but should have two of them, one for mup4test and one for vaf

@rezbyte
Copy link
Contributor

rezbyte commented Mar 28, 2024

Good catch, but the color was enabled on the web. Could be just a server delay?

So if I am correct, you had the both the web and Android clients open at the same time. The web client with the custom colors and the Android without. Is that right?

If that is the case, I do not think it is related to the onboarding wizard. The Android client was respecting the settings it had. This might be an issue about syncing user settings.

Yes, but should have two of them, one for mup4test and one for vaf

If the custom colors were disabled on the Android client, the vaf theme would not appear.

@murilopereirame
Copy link
Contributor Author

murilopereirame commented Mar 28, 2024

So if I am correct, you had the both the web and Android clients open at the same time. The web client with the custom colors and the Android without. Is that right?

If that is the case, I do not think it is related to the onboarding wizard. The Android client was respecting the settings it had. This might be an issue about syncing user settings.

No, I setup the whitelabel on the web, put a nice color on it and after that I added the account to the android device and the theme wasn't shown up.

If the custom colors were disabled on the Android client, the vaf theme would not appear.

Yes, probably is related to the last question. Also the vaf theme took a while to appears into the wizard, I had to trigger the wizard 2/3 times

To be honest, maybe isn't a thing with the wizard itself, but with the load process of custom themes.

@charlag
Copy link
Contributor

charlag commented Apr 2, 2024

I mentioned it but maybe only in person but the wizard page grabs only what is already loaded:
https://github.com/tutao/tutanota/pull/6785/files#diff-9bb05348e2e34585caf2fecc84648f110a3d340be0bc1f5b26c7cbe2e9a167dfR14

so it's probably a timing issue, whether PostLoginActions will add a theme first or not. We should probably make sure we do run storeNewCustomThemes() (or some alternative) when showing the wizard

@wrdhub wrdhub added the state:done meets our definition of done label Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android issues that concern the android client but not all clients dev bug unpublished bugs, found during our development/test cycle (excluded from release notes) state:done meets our definition of done
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants