-
Notifications
You must be signed in to change notification settings - Fork 0
Added internal store for tasks #5
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get this garbage out of my repository.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to be gitignored
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here
@@ -1,9 +1,14 @@ | |||
package masecla.remindify.dto; | |||
|
|||
import jakarta.persistence.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather not have a star import for two annotations.
spring.datasource.url=jdbc:h2:file:./data/remindify-new | ||
spring.datasource.driver-class-name=org.h2.Driver | ||
spring.datasource.username=username | ||
spring.datasource.password=password | ||
|
||
spring.jpa.hibernate.ddl-auto=update | ||
|
||
spring.h2.console.enabled=true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please document how these are meant to be used / switch from local to remote mysql
TaskDto persisted = task.getId() == null ? taskRepository.save(task) : task; | ||
log.info("Task with id {} has been created.", persisted.getId()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make a TaskService and call it from here, I don't want the consumer service to touch the dto repo
Fixes #2 |
bonjour ampù l'amicu