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

INT-889 Create Oauth page #2

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

fuboki10
Copy link
Collaborator

PR Description

What this PR does

  • User friendly index page

How to manually test

  • APP_CONFIG=config/google.js yarn start

Reviewer Checklist (to be filled by reviewer)

Acceptance Criteria

  • It is possible to understand the code without explanations.
  • "Statement coverage" of the source code: >=70%. Deviation:
  • All non-trivial methods are documented with docstrings.
  • Names of classes, variables, methods, etc. follow naming conventions and are self-describing.
  • Code does not contain hard-coded paths or labels.
  • Errors are handled as close and specific as possible.
  • Code reflects the architecture.
  • Code reflects the specification.

@fuboki10 fuboki10 self-assigned this Apr 26, 2024
@fuboki10 fuboki10 marked this pull request as ready for review April 26, 2024 01:03
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.

1 participant