Skip to content
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

Refactor: create middleware which set the org for the apis dedicated to orgs #4207

Closed
6543 opened this issue Oct 7, 2024 · 1 comment · Fixed by #4208
Closed

Refactor: create middleware which set the org for the apis dedicated to orgs #4207

6543 opened this issue Oct 7, 2024 · 1 comment · Fixed by #4208
Labels
good first issue Likely to be an easy fix refactor delete or replace old code

Comments

@6543
Copy link
Member

6543 commented Oct 7, 2024

currently each api endpoint function has the org-id parsing and loading from store defined on the start.

similar to how there is a middleware for repos witch use session.Repo(c) we should do the same for organisazions ...
@6543 6543 added refactor delete or replace old code good first issue Likely to be an easy fix labels Oct 7, 2024
@6543 6543 mentioned this issue Oct 7, 2024
Merged
@6543
Copy link
Member Author

6543 commented Oct 7, 2024

well in the end i created a pull: #4208

@6543 6543 closed this as completed in #4208 Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Likely to be an easy fix refactor delete or replace old code
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use middleware to load org 6543-forks/woodpecker
1 participant
@6543