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

fix: reimpliment highlight functionality on rate limit #79

Merged
merged 2 commits into from
Aug 19, 2024

Conversation

jordan-ae
Copy link
Contributor

Resolves #78
Screenshot from 2024-08-18 18-56-39

@jordan-ae jordan-ae requested a review from 0x4007 as a code owner August 18, 2024 18:10
@ubiquity-os-deployer
Copy link

ubiquity-os-deployer bot commented Aug 18, 2024

@0x4007
Copy link
Member

0x4007 commented Aug 18, 2024

@gentlementlegen This cypress error seems to reoccur intermittently. Can you look into this? Doesn't seem like a good idea to merge commits that aren't passing CI

Copy link
Member

@0x4007 0x4007 left a comment

Choose a reason for hiding this comment

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

I can't git blame easily from mobile but curious why this was removed in the first place.

@jordan-ae
Copy link
Contributor Author

@0x4007 it looks like it was deleted by you when you were re-implementing error toast

@gentlementlegen
Copy link
Member

gentlementlegen commented Aug 18, 2024

@0x4007 This test has been failing for quite some time but it seems that PRs went in regardless. I think it's one line of code to remove in the test, because the bottom bar does not scroll anymore (got changed when FAQ button was added I believe). I will commit in this PR.


Should be good to go now. Also tested the last build, the login button shines on desktop and mobile on rate limitation.

@0x4007 0x4007 merged commit 803107b into ubiquity:development Aug 19, 2024
2 checks passed
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.

Rate Limited Login Button No Longer Highlighted
3 participants