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

DEV-3477-animate-on-the-contributions-dashboard-upgrade-prompt #1134

Merged

Conversation

GuiMend
Copy link
Contributor

@GuiMend GuiMend commented May 26, 2023

Please fill out each section even if it's just with "N/A"

Plan? (if this draft/incomplete indicate what you intend to do and how)

What's this PR do?

Animate upgrade prompt show that it's more visible to the users.

Why are we doing this? How does it help us?

We want to make the upgrade prompt more visible for users.

How should this be manually tested? Please include detailed step-by-step instructions.

  • Log in as a Free user, connect to stripe and publish a page
  • Go in and out of the Contributions dashboard page
  • You should see the prompt animating

Did this PR make changes to the user interface that may require changes to the user-facing docs?

No

Have automated unit tests been added? If not, why?

Yes

How should this change be communicated to end users?

N/a

Are there any smells or added technical debt to note?

no

Has this been documented? If so, where?

no

What are the relevant tickets? Add a link to any relevant ones.

https://news-revenue-hub.atlassian.net/browse/DEV-3477

Do any changes need to be made before deployment to production (adding environment variables, for example)?

no

Are there next steps? If so, what? Have tickets been opened for them? List next-step tickets here:

no

@GuiMend GuiMend requested a review from nrh-cklimas as a code owner May 26, 2023 15:58
@x110dc x110dc temporarily deployed to rev-engine-dev-3477-ani-9cqtyb May 26, 2023 16:00 Inactive
@x110dc x110dc temporarily deployed to rev-engine-dev-3477-ani-9cqtyb May 29, 2023 20:41 Inactive
@cypress
Copy link

cypress bot commented May 29, 2023

Passing run #5359 ↗︎

0 210 1 0 Flakiness 0

Details:

clearTimeout + fix skipped tests
Project: NRE Commit: 04fce12cd3
Status: Passed Duration: 07:04 💡
Started: May 30, 2023 11:01 PM Ended: May 30, 2023 11:08 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@x110dc x110dc temporarily deployed to rev-engine-dev-3477-ani-9cqtyb May 29, 2023 21:09 Inactive
@github-actions
Copy link

github-actions bot commented May 29, 2023

Jest unit test coverage

Lines Statements Branches
69.7% 69% 61.8%

@x110dc x110dc temporarily deployed to rev-engine-dev-3477-ani-9cqtyb May 30, 2023 22:59 Inactive
@x110dc x110dc merged commit 14d096b into main Jun 5, 2023
12 checks passed
@x110dc x110dc deleted the DEV-3477-animate-on-the-contributions-dashboard-upgrade-prompt branch June 5, 2023 23:41
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.

None yet

3 participants