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

Adjusts on text when the Upgrade to Gold is unavailable #5598

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

TheKoakuma
Copy link

@TheKoakuma TheKoakuma commented Nov 19, 2023

Based on the comment from unitooni on forum #21157 ,those commits are very straightforward, making some changes on the view when upgrade (a.k.a Gold) is unavailable.

Also (partially) Fixes #5265

@pokeball99
Copy link
Contributor

Personally i feel like it would better to implement a way of toggling upgrade membership purchase rather than hardcoding it like this one way or another

@TheKoakuma
Copy link
Author

TheKoakuma commented Nov 19, 2023

Personally i feel like it would better to implement a way of toggling upgrade membership purchase rather than hardcoding it like this one way or another

As far as I could study the code, there is a toggle, but only the upgrade page were using the flag that states this, I just made the changes on Users and Posts pages to react to UserUpgrade.enabled too.

@TheKoakuma
Copy link
Author

@nonamethanks @evazion is there any necessary changes to my PR?

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.

Handle Gold upgrades not being available
2 participants