-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: endpoint challenge #33
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, remember that the branch name should start with a semantic type like feat/
, fix/
, etc.
OMG!, You're absolutely right, I overlooked it, it won't happen again |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests are missing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Squash and merge .
You can start using this in the React client, but keep in mind that we will need to create another pull request to follow the .net proxy specification.
* docs: OpenApi Specification (#9) * chore: Update docker-compose (#12) * chore: Add missing env variables and services to docker-compose * chore: Use worker default env variables * ci: Linter and formatter (#14) * chore: Add recommended vscode extensions * chore(deps): Install black formatter and ruff linter * style: Lint and format files * docs: Add linting and formatting commands to README * ci: Add linter and style checks to integration pipeline * feat: endpoint login (#15) * feat: endpoint challenge (#33) --------- Co-authored-by: Antonio Donis <[email protected]> Co-authored-by: Pedro Andrés Chaparro Quintero <[email protected]>
Closes #21