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

refactor comment with DDD #150

Merged
merged 4 commits into from
May 19, 2024
Merged

refactor comment with DDD #150

merged 4 commits into from
May 19, 2024

Conversation

TremblingV5
Copy link
Owner

Link to an issue

What

refactor comment with DDD

How

Totally, there're 4 directories in comment: api, domain, infra, services. I try my best to follow the ideas of DDD in development.

I supposed that only keep comment as a point for practise DDD. DDD it not 100% suitable for DouTok right now.

Screenshots

How to test

Checklist

  • Link to an issue if it really related.
  • At least describe what this PR does.
  • Use rebase to confirm that current branch doesn't conflict with main branch.
  • Unit tests. At least do not reduce the unit test coverage.
  • Checked and updated guidelines.md which used to describe how to build, deploy and use DouTok.

@TremblingV5 TremblingV5 force-pushed the refactor_comment_ddd branch from bbd273b to 6785aa6 Compare April 17, 2024 12:08
@TremblingV5 TremblingV5 merged commit e3eceb9 into main May 19, 2024
0 of 2 checks passed
linjiang82 added a commit to linjiang82/DouTok that referenced this pull request May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant