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.

Sign up for GitHub

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

Jump to bottom

Merge/repository token api #744

Merged
merged 4 commits into from
Aug 26, 2022
Merged

Merge/repository token api #744

merged 4 commits into from
Aug 26, 2022

Conversation

victor-salles
Copy link
Contributor

  • Adds a new repository/authorization-by-user API which allows for an authenticated user to gt their token in a given Repository
  • Adds TEST_REPOSITORY_ID environment variable

@victor-salles
Create repository/authorization-by-user route
002f4e7 
Creates a RepositoryTokenByUserViewSet which retrieves the user access
token based on a new environment variable named TEST_REPOSITORY_ID.
@victor-salles
Fix TEST_REPOSITORY_ID default
5b1ee93
@victor-salles
Fix/ Add serializer_class to RepositoryTokenByUser
ca60842
@victor-salles
Fix/ remove env variable from docker-compose
25014e4
@sonarcloud
Copy link

sonarcloud bot commented Aug 26, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@coveralls
Copy link

Pull Request Test Coverage Report for Build 4688

  • 6 of 19 (31.58%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.2%) to 81.072%
Changes Missing Coverage Covered Lines Changed/Added Lines %
bothub/api/v2/repository/views.py 4 17 23.53%
Totals Coverage Status
Change from base Build 4676: -0.2%
Covered Lines: 4660
Relevant Lines: 5748
💛 - Coveralls

@victor-salles victor-salles merged commit d9b6947 into master Aug 26, 2022
@victor-salles victor-salles deleted the merge/repository-token-api branch August 26, 2022 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sandro-Meireles Sandro-Meireles Sandro-Meireles approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@victor-salles @coveralls @Sandro-Meireles