Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

reapply session auth changes #450

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Conversation

TangoYankee
Copy link
Member

Checklist

  • Add description
  • Reference the open issue that the pull request addresses
  • Pass code quality checks
    • spin up docker docker-compose up -d --build
    • enter api container docker-compose exec api /bin/bash
    • run api tests make validate
    • exit container ctrl/command+D or exit
    • enter web container docker-compose exec web /bin/sh
    • run front-end tests npm run test or npx jest
    • lint npm run lint-fix
    • exit container as above
  • Request code review
    • Please allow 36 hours from opening a pull request before merging a pull request- even if it has already received an approving review.
  • Address comments on code and resolve requested changes
  • Merge own code

Description

Issue: #

Brief description of solution

@TangoYankee
Copy link
Member Author

Blocked by #448

@TangoYankee TangoYankee added the status: blocked Progress can't be made until a dependency is resolved label Apr 2, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
status: blocked Progress can't be made until a dependency is resolved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant