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

[MERGE WITH GIT FLOW] Hotfix/4538 fix api key signup load #4557

Merged
merged 4 commits into from
Aug 14, 2020

Conversation

jason-upchurch
Copy link
Contributor

@jason-upchurch jason-upchurch commented Aug 13, 2020

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:

  • clear browser history
  • check out master branch
  • run local api, see if form loads, try refreshing until form does not load
  • clear browser history (again)
  • check out this hotfix branch
  • run local api
  • test that api key signup form loads with page, refresh, etc.

Impacted areas of the application

List general components of the application that this PR will affect:

  • rest.py
  • swagger-ui.html
  • docs.py

@jason-upchurch jason-upchurch added Bug Hotfix Merge with git flow labels Aug 13, 2020
@jason-upchurch jason-upchurch added this to the Sprint 13.2 milestone Aug 13, 2020
@jason-upchurch jason-upchurch changed the title Hotfix/4538 fix api key signup load [MERGE WITH GIT FLOW] Hotfix/4538 fix api key signup load Aug 13, 2020
Copy link
Contributor

@fec-jli fec-jli 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 reproduce the bug locally. after checking in the hotfix branch, re-try many times, not appear any more. great work. Thanks

@jason-upchurch jason-upchurch force-pushed the hotfix/4538-fix-api-key-signup-load branch from b7bc04b to 9b2d047 Compare August 13, 2020 19:36
@jason-upchurch
Copy link
Contributor Author

jason-upchurch commented Aug 13, 2020

@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/

@jason-upchurch
Copy link
Contributor Author

@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 :)
cc @patphongs

@johnnyporkchops
Copy link
Contributor

We are using. a setTimeout to ensure the form is loaded every time. While not ideal, it seems to solve the issue. I was hesitant to suggest this solution before, but feel a little more confident with the sign-off of another frontend dev.

Copy link
Contributor

@JonellaCulmer JonellaCulmer left a 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.

@jason-upchurch jason-upchurch requested review from patphongs and removed request for rfultz August 14, 2020 12:42
@lbeaufort lbeaufort merged commit b301246 into master Aug 14, 2020
@lbeaufort lbeaufort deleted the hotfix/4538-fix-api-key-signup-load branch September 15, 2020 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Hotfix Merge with git flow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants