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

Add DEI Badging endpoints #2444

Merged
merged 6 commits into from
Jun 22, 2023
Merged

Add DEI Badging endpoints #2444

merged 6 commits into from
Jun 22, 2023

Conversation

Ulincsys
Copy link
Contributor

Description

  • Start work on DEI badging
  • Finish track repo endpoint
  • Complete testing on DEI endpoint
  • Testing for placeholder report generation

Signed commits

  • Yes, I signed my commits.

- Add DEI endpoints
 - Unfinished ATM
- Add BadgingDEI table

Signed-off-by: Ulincsys <[email protected]>
- Switch to DatabaseSession for object management
- Edit alembic revision with auto-increment
- Fix logic errors

Co-authored-by: IsaacMilarky <[email protected]>

Signed-off-by: Ulincsys <[email protected]>
- Lowercase provided repo_url in dei/repo/add
- The dei/report endpoint now returns an error if the given
  project ID does not exist
- Add mdpdf as dependency for generating report PDFs
- Add kwargs to DatabaseSession
- Add dei badging report markdown template

Signed-off-by: Ulincsys <[email protected]>
@sgoggins
Copy link
Member

@ABrain7710 : Can you check the DB stuff tonight?

@sgoggins
Copy link
Member

@Ulincsys / @ABrain7710
@Ulincsys : Could you create a simple markdown doc explaining how to USE the API, with some examples, and what the endpoints are specifically?

@ABrain7710 / @Ulincsys : Could we get the same for login. They are related and I actually don't know how to test any of this myself without that info. :)

- Add documentation for Augur Oauth endpoints
 - user/session/generate
 - user/session/refresh
- Add documentation for DEI endpoints
 - dei/repo/add
 - dei/report
- Fix consistency issues in user endpoints
- Add documentation describing the Augur Oauth flow

Signed-off-by: Ulincsys <[email protected]>
@sgoggins sgoggins merged commit 3be6797 into dev Jun 22, 2023
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.

2 participants