Skip to content

Conversation

@mateo-ivc
Copy link
Collaborator

@mateo-ivc mateo-ivc commented Nov 3, 2025

Description

Moving routes and api files into own packages (user/sessions) based on PR#5510

Changelog

  • expand initializer package by api- & routes initializer
  • moving api.Context.go into own package -> middleware
  • moving middleware that need a service into designated package
  • each package has not it's own router -> will later be joined together to a single big router
  • added api tests

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • The light- and dark-theme are both supported and tested
  • The design was implemented and is responsive for all devices and screen sizes
  • The application was tested in the most commonly used browsers (e.g. Chrome, Firefox, Safari)

(Optional) Visual Changes

@mateo-ivc mateo-ivc marked this pull request as draft November 3, 2025 10:39
@mateo-ivc mateo-ivc marked this pull request as ready for review November 10, 2025 14:32
Base automatically changed from mi/split-user-and-sessions to main November 17, 2025 13:35
@mateo-ivc mateo-ivc force-pushed the refactor/api-refactoring branch from f3d6b90 to 003f1c0 Compare November 17, 2025 13:54
@github-actions
Copy link

The deployment to the dev cluster was successful. You can find the deployment here: https://5563.development.scrumlr.fra.ics.inovex.io
This deployment is only for testing purposes and will be deleted after 1 week.
To redeploy rerun the workflow.
DO NOT STORE IMPORTANT DATA ON THIS DEPLOYMENT

Deployed Images
  • ghcr.io/inovex/scrumlr.io/scrumlr-frontend:sha-1047ad4

  • ghcr.io/inovex/scrumlr.io/scrumlr-server:sha-1047ad4

@mateo-ivc mateo-ivc added this pull request to the merge queue Nov 24, 2025
Merged via the queue into main with commit 1daed00 Nov 24, 2025
18 of 19 checks passed
@mateo-ivc mateo-ivc deleted the refactor/api-refactoring branch November 24, 2025 13:59
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.

3 participants