Skip to content

1132 - notification engine implementation + jobStatus email notifications #2337

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
Merged
merged 26 commits into from
Apr 14, 2025
Merged

1132 - notification engine implementation + jobStatus email notifications #2337

merged 26 commits into from
Apr 14, 2025

Conversation

matijapetanjek
Copy link
Collaborator

Still to do:

  • WEBHOOK jobStatus notifications
  • frontend form for notification configuration

@matijapetanjek matijapetanjek force-pushed the 1132 branch 3 times, most recently from 81b8c1e to 43f3082 Compare April 4, 2025 18:30
@matijapetanjek matijapetanjek requested a review from ivicac April 4, 2025 19:27
@matijapetanjek matijapetanjek force-pushed the 1132 branch 5 times, most recently from d45b394 to 833c79f Compare April 8, 2025 16:18
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Apr 8, 2025

Quality Gate Passed Quality Gate passed for 'server'

Issues
41 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@matijapetanjek matijapetanjek marked this pull request as ready for review April 8, 2025 19:02
matijapetanjek and others added 18 commits April 14, 2025 06:21
@matijapetanjek @ivicac
1132 - typo
949f3ba
@matijapetanjek @ivicac
1132 define notifications DB schemas
45debbe
@matijapetanjek @ivicac
1132 define notification entities
7ce9461
@matijapetanjek @ivicac
1132 init notification repository and service
5fd31b3
@matijapetanjek @ivicac
1132 initial test configuration
47f2928
@matijapetanjek @ivicac
1132 remove custom application event listeners outside atlas engine
1d0836a
@matijapetanjek @ivicac
1132 add interfaces that will be used to define notification content
b869d26
@matijapetanjek @ivicac
1132 add classes for sending email and webhook notifications
7c6d81b
@matijapetanjek @ivicac
1132 implement email notifications for Job execution statuses
5ca88d3
@matijapetanjek @ivicac
1132 add Event Listener for JobStatus notifications
515da68
@matijapetanjek @ivicac
1132 update openapi spec with new endpoints (will add update later)
1c4b3c0
@matijapetanjek @ivicac
1132 implement Notification and Event Controllers
025fdf7
@matijapetanjek @ivicac
1132 extract mailService to platform-mail module
2130ae2
@matijapetanjek @ivicac
1132 refactor code with platform-mail module
3b928be
@matijapetanjek @ivicac
1132 generateOpenAPI - server
651dd18
@matijapetanjek @ivicac
1132 generateOpenAPI - client
e2d8106
@matijapetanjek @ivicac
1132 npm run format
90d5eff
@ivicac
1132 - Reorganize, SF
a8d297f
@ivicac ivicac merged commit f6eceea into master Apr 14, 2025
4 of 5 checks passed
@ivicac ivicac deleted the 1132 branch April 14, 2025 07:44
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed for 'client'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ivicac ivicac Awaiting requested review from ivicac

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@matijapetanjek @ivicac