Skip to content

Conversation

serban26
Copy link
Collaborator

bonjour ampù l'amicu

@serban26 serban26 requested a review from masecla22 September 17, 2025 20:06
@serban26 serban26 self-assigned this Sep 17, 2025
Copy link
Contributor

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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same

Copy link
Contributor

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

Copy link
Contributor

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.*;
Copy link
Contributor

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.

Comment on lines +4 to +11
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
Copy link
Contributor

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

Comment on lines 27 to 28
TaskDto persisted = task.getId() == null ? taskRepository.save(task) : task;
log.info("Task with id {} has been created.", persisted.getId());
Copy link
Contributor

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

@masecla22 masecla22 changed the title Solve issues Added internal store for tasks Sep 17, 2025
@masecla22
Copy link
Contributor

Fixes #2

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.

2 participants