Skip to content

Deployment#12

Closed
meowoverlord wants to merge 7 commits intomainfrom
deployment
Closed

Deployment#12
meowoverlord wants to merge 7 commits intomainfrom
deployment

Conversation

@meowoverlord
Copy link
Copy Markdown

pls review
Es kann sein das mein main local anders ist und dadurch unerwünschter Code gepusht wurde. Bitte alles überprüfen!

@meowoverlord meowoverlord marked this pull request as draft September 22, 2025 08:30
@meowoverlord meowoverlord marked this pull request as ready for review September 22, 2025 08:31
@meowoverlord meowoverlord reopened this Sep 22, 2025
davidclarafigueiredo pushed a commit that referenced this pull request Oct 17, 2025
* add endpoint to get last 5 notifications from user

* add cors config for frontend

* restrict prod origins

* remove placeholder authentication

* remove unused import

* quick test

* fix tests

* update basepath for deployment

* change prod base url

* test deployment

* test deployment

* test deployment

* test deployment

* test deployment

* don't deploy on every push

* fix linting

* move userId and notificationId into request body

* move notificationId into request path

* move notificationId into request path

* fix linting

* move userId into requestparam

* remove paging

* use spring actuator for readiness and liveness probes

* fix linting

* return 201 on notification creation

* return bad responses code via errors

* add disclaimer

* fix linting
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