-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat(invitation): allow access to invitation endpoint to technical user #933
feat(invitation): allow access to invitation endpoint to technical user #933
Conversation
4ab2608
to
fcea7eb
Compare
@typecastcloud please update the dependencies. can be found in the artefacts here: https://github.com/eclipse-tractusx/portal-backend/actions/runs/10402110437?pr=933 |
fcea7eb
to
070cb92
Compare
Hi @Phil91, I rebased to release/v2.2.0-RC4 branch. Tests are failing on that branch. Not sure if that also resolved dependencies. |
Hi @typecastcloud I suggest to do an interactive rebase dropping all commits that aren't yours |
Technical user/clients were now able to use endpoint. Refs: eclipse-tractusx#932
070cb92
to
ed47439
Compare
Done. Rebased on RC5. Not sure why this PR wants updated dependencies. |
@evegufy I also updated dependencies now I didn't notice that this was considered for 2.2.0. |
@MaximilianHauer should this be included in the 2.2.0 release? |
@typecastcloud I didn't question the base branch up to now but yes, I'd prefer to move it to main, not to 2.2.0. |
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MaximilianHauer could you please review as well?
Just saw #932 (comment) |
I'll create a separate PR for this since it requires more complex additions to the code base. |
Description
Allow access to invitation endpoint to technical user.
Why
Technical user/clients were not able to use endpoint.
Issue
Refs: #932
Checklist