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.

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

chore: campaign application create contract #648

Merged

Conversation

gparlakov
Copy link
Contributor

Copy link

github-actions bot commented Jun 23, 2024

✅ Tests will run for this PR. Once they succeed it can be merged.

@gparlakov gparlakov force-pushed the campaign-application-contract branch from c97776f to 4ac8829 Compare June 23, 2024 10:57
Copy link
Member

@sashko9807 sashko9807 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.
Just for reference @AuthenticatedUser decorator, is only needed if we want to extract data from the AccessToken(e.g. keycloakId of the user making the request) which will be used for something. Otherwise it is not necessary.

@sashko9807 sashko9807 added the run tests Allows running the tests workflows for forked repos label Jun 23, 2024
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Jun 23, 2024
@sashko9807 sashko9807 merged commit 7dc9b9b into podkrepi-bg:master Jun 23, 2024
10 of 11 checks passed
gparlakov added a commit to gparlakov/podkrepi-bg-api that referenced this pull request Jun 30, 2024
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