-
Notifications
You must be signed in to change notification settings - Fork 105
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
[MERGE WITH GIT FLOW] Hotfix/4538 fix api key signup load #4557
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can reproduce the bug locally. after checking in the hotfix branch, re-try many times, not appear any more. great work. Thanks
b7bc04b
to
9b2d047
Compare
9b2d047
to
79c0216
Compare
@JonellaCulmer I added you as a reviewer because we wanted to make sure the operation of the api signup form loading was acceptable from a usability standpoint. Thanks for your review! It's still in the process of deploying, but when done, it will be available for review: https://fec-dev-api.app.cloud.gov/developers/ |
@rfultz thank you for your tremendous support today with tweaking the js to get the form loading consistently! I really appreciate your collaboration on this effort and your help in all of our success :) |
We are using. a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested across two browsers and incognito mode and no issues found. The form now loads consistently each time. Some slight delays, but nothing severe.
Summary (required)
Include a summary of proposed changes.
How to test the changes locally
This bug is sometimes hard to reproduce. But @johnnyporkchops and @jason-upchurch confirmed we were able to reproduce on local. So:
Impacted areas of the application
List general components of the application that this PR will affect: