fixup! Merge pull request 'Notifications on Schedule' (#172) from NotificationSchedule into master
Some checks failed
Build and test backend / Build-backend (push) Successful in 1m26s
deploy to production / deploy-frontend (push) Successful in 27s
deploy to production / deploy-backend (push) Successful in 54s
Build and test FrontEnd / Build-frontend (push) Has been cancelled

This commit is contained in:
Debucquoy Anthony 2024-04-22 00:03:45 +02:00
parent 71c2af7fcb
commit bbddcb26ad
Signed by: tonitch
GPG Key ID: A78D6421F083D42E

View File

@ -8,6 +8,7 @@ import ovh.herisson.Clyde.Repositories.Msg.ForumRepository;
import ovh.herisson.Clyde.Repositories.Msg.TopicRepository;
import ovh.herisson.Clyde.Services.UserService;
import ovh.herisson.Clyde.Tables.Course;
import ovh.herisson.Clyde.Tables.Notification;
import ovh.herisson.Clyde.Tables.User;
import ovh.herisson.Clyde.Tables.Msg.Answer;
import ovh.herisson.Clyde.Tables.Msg.Forum;
@ -25,7 +26,7 @@ public class ForumService {
public void createForum(Course c, Forum f){
c.addForum(f);
for (User u: f.getRegister()) {
userServ.Notify(u, new Notification("forum.notification.forum.new", data.getSubject(), "/#/Forum"));
userServ.Notify(u, new Notification("forum.notification.forum.new", f.getName(), "/#/Forum"));
}
courseRepo.save(c);
}