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

Strategy 패턴 적용 #118

Merged
merged 12 commits into from
Dec 9, 2023
Merged

Strategy 패턴 적용 #118

merged 12 commits into from
Dec 9, 2023

Conversation

yjeong-k
Copy link
Contributor

@yjeong-k yjeong-k commented Dec 9, 2023

PR Title: Strategy 패턴 적용

Related Issue(s):

PR Description:

EmailVerificationViewModel에 strategy 패턴을 적용하였습니다
자세한 사항은 제출할 design pattern document 참고해주시면 감사하겠습니다

Changes Included:
  • Added new feature(s)
  • Fixed identified bug(s)
  • Updated relevant documentation
Notes for Reviewer:

Any specific instructions or points to be considered by the
reviewer.

Reviewer Checklist:

  • Code is written in clean, maintainable, and idiomatic form.
  • Automated test coverage is adequate.
  • All existing tests pass.
  • Manual testing has been performed to ensure the PR works as expected.
  • Code review comments have been addressed or clarified.

Additional Comments:

Add any other comments or information that might be useful for the
review process.

@yjeong-k yjeong-k added the frontend Feature implementation for frontend label Dec 9, 2023
@yjeong-k yjeong-k requested a review from JH747 December 9, 2023 16:10
@JH747 JH747 merged commit d048056 into main Dec 9, 2023
1 check passed
@JH747 JH747 deleted the refactor/strategy branch December 9, 2023 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Feature implementation for frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants