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.

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

find a better name for repohook #8

Open
lucagiove opened this issue Jan 10, 2024 · 4 comments
Open

find a better name for repohook #8

lucagiove opened this issue Jan 10, 2024 · 4 comments
Milestone
0.1.0

Comments

@lucagiove
Copy link
Contributor

No description provided.

@lucagiove lucagiove converted this from a draft issue Jan 10, 2024
@lucagiove lucagiove added this to the 0.1.0 milestone Jan 10, 2024
@lucagiove
Copy link
Contributor Author

Probably something with that refers to transaction?

@lucagiove
Copy link
Contributor Author

if we want to change this I think we should discuss it asap otherwise let's keep this name @gtoselli

@lucagiove
Copy link
Contributor Author

  1. RepoTransactionalHook
  2. OnSaveHook
  3. ...

Actually since the only implemented method is onSave might be correct to underline the concept

@lucagiove
Copy link
Contributor Author

if we implement delete or other methods probably the name is correct since we might have multiple onMethods

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
ddd-toolkit
Status: Todo
Milestone
0.1.0
Development

No branches or pull requests
1 participant
@lucagiove