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

refine api key signup page #4385

Merged
merged 3 commits into from
Jul 16, 2020
Merged

Conversation

jason-upchurch
Copy link
Contributor

@jason-upchurch jason-upchurch commented Jun 1, 2020

Summary (required)

refine API key signup elements per completion criteria in #4363

How to test the changes locally

for non developers:

for developers:

Impacted areas of the application

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

  • swagger-ui.html
  • docs.py

@jason-upchurch jason-upchurch force-pushed the feature/4363-api-signup-refinement branch from b7de3e3 to ef342e6 Compare June 23, 2020 20:58
@jason-upchurch jason-upchurch force-pushed the feature/4363-api-signup-refinement branch from 2b0fcd7 to 90097b7 Compare July 2, 2020 18:26
@jason-upchurch jason-upchurch changed the title [WIP] refine api key signup page refine api key signup page Jul 2, 2020
@jason-upchurch jason-upchurch self-assigned this Jul 2, 2020
Copy link
Member

@lbeaufort lbeaufort left a comment

Choose a reason for hiding this comment

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

Thanks for your work on this, @jason-upchurch! Before merge, can you remove the extra dev deploy?

openFEC/tasks.py

Lines 145 to 146 in 90097b7

('dev', lambda _, branch: branch == 'develop'),
('dev', lambda _, branch: branch == 'feature/4363-api-signup-refinement')

@jason-upchurch
Copy link
Contributor Author

Thanks for your work on this, @jason-upchurch! Before merge, can you remove the extra dev deploy?

openFEC/tasks.py

Lines 145 to 146 in 90097b7

('dev', lambda _, branch: branch == 'develop'),
('dev', lambda _, branch: branch == 'feature/4363-api-signup-refinement')

@lbeaufort we don't want to merge the dev deploy code in. That's there for the benefit of anyone wanting to review on dev. @JonellaCulmer @johnnyporkchops @pkfec can you please make a note in the issue if you do not plan to review? I will make one last amended commit before merge to remove the dev deploy (or redeploy to dev should any reviewers want to inspect my work on dev).

@JonellaCulmer
Copy link
Contributor

@jason-upchurch I don't see any of the changes reflected in the https://fec-dev-api.app.cloud.gov/developers/ link you provided.

@jason-upchurch
Copy link
Contributor Author

@jason-upchurch I don't see any of the changes reflected in the https://fec-dev-api.app.cloud.gov/developers/ link you provided.

@JonellaCulmer that's because dev has been overwritten many times since I deployed this I think a week or so ago. I'll check with the team and redeploy it for you.

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.

Looks good. Thanks @jason-upchurch

@jason-upchurch jason-upchurch force-pushed the feature/4363-api-signup-refinement branch from 90097b7 to 3aa8bae Compare July 16, 2020 14:31
Copy link
Contributor

@pkfec pkfec left a comment

Choose a reason for hiding this comment

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

Looks good @jason-upchurch.

Copy link
Contributor

@johnnyporkchops johnnyporkchops left a comment

Choose a reason for hiding this comment

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

Looks great!, jut a few minor CSS comments.

webservices/templates/swagger-ui.html Outdated Show resolved Hide resolved
webservices/templates/swagger-ui.html Outdated Show resolved Hide resolved
@jason-upchurch jason-upchurch force-pushed the feature/4363-api-signup-refinement branch from 3aa8bae to 94abd87 Compare July 16, 2020 16:58
@lbeaufort lbeaufort merged commit edceeff into develop Jul 16, 2020
@lbeaufort lbeaufort deleted the feature/4363-api-signup-refinement branch July 30, 2020 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorporate design/UX and feature flag for API key sign up
5 participants