Skip to content

feat(auth): add base spring security #94

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
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat(auth): add base spring security #94

wants to merge 1 commit into from
+586 −41

Conversation

mannn-2274
Copy link
Contributor

@mannn-2274 mannn-2274 commented Apr 7, 2024 •
edited
Loading

Summary

  • add base spring security

Description

  • add base spring security

Checklist

  • 1. Build successfully on local
  • 2. Format all code files included in the PR
  • 3. No more unconfirmed specs in the PR
  • 4. Tested the normal case and error cases following specs
  • 5. Is there a refactor to the old code? If yes, have you tested the old function and noted the impact yet?
  • 6. Loops have a set length and correct termination conditions
  • 7. There aren't any redundant or duplicate codes in PR (clean code)
  • 8. There aren't any hard-coded in the code in PR
  • 9. Optimization logic handling and optimizations SQL (for example, don't get redundant data or add an index to optimize performance, query only necessary fields...)
  • 10. Set pull-request title by format: Feature new #{id}: {short des}, Bug fix #{id}: {short des}
  • 11. Update issue

Evidence

NA

Discussion

OK

@mannn-2274 mannn-2274 self-assigned this Apr 7, 2024
@mannn-2274 mannn-2274 added the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@khuongln-1346 khuongln-1346 Awaiting requested review from khuongln-1346

@haidakto321 haidakto321 Awaiting requested review from haidakto321

@hoangntv-1399 hoangntv-1399 Awaiting requested review from hoangntv-1399

At least 1 approving review is required to merge this pull request.

Assignees

@mannn-2274 mannn-2274

Labels
type: feature Brand new functionality, features, pages, workflows, endpoints, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mannn-2274