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

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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