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

API Create company account | Add invitation id and api for invitation details #834

Closed
SanaSalem01 opened this issue Jul 15, 2024 · 8 comments
Assignees

Comments

@SanaSalem01
Copy link

SanaSalem01 commented Jul 15, 2024

Change proposal:
Add Company Id,application id in response of ExecuteInvitation api i.e POST: api/administration/invitation

Reason:
In automation and CRM communication there was no reference id to track the Invitation.

Current Impact/limitations:

  • The missing response information makes implementing APIs and integration tests more time-consuming.
  • CRM communication there was no reference id to track the Invitation

Change proposal impact:
So now any system can use Application id and Company Id for reference after invitation.

@jjeroch
Copy link
Contributor

jjeroch commented Jul 15, 2024

@SanaCofinityX please update the ticket - this is a change proposal and not a bug (as it intends when reading the title" please use something like "API Create company account | adding "applicationID" in the response body" this makes it more open. Please also share the info if you want to contribute the code for it.

@SanaSalem01 SanaSalem01 changed the title Invite/Register company | Information in response logs is missing. API Create company account | Add "applicationID" in the response body Jul 16, 2024
@SanaSalem01
Copy link
Author

@jjeroch
Ticket summary has been updated.

@tfjanjua
Copy link
Contributor

@jjeroch | @evegufy
I would like to ask from which release branch I should checkout for this change!
I am assuming it would be https://github.com/eclipse-tractusx/portal-backend/tree/release/v2.1.0-RC1 but please let me know if there is any other release to checkout.

Thanks

@evegufy
Copy link
Contributor

evegufy commented Jul 22, 2024

@jjeroch | @evegufy
I would like to ask from which release branch I should checkout for this change!
I am assuming it would be https://github.com/eclipse-tractusx/portal-backend/tree/release/v2.1.0-RC1 but please let me know if there is any other release to checkout.

Thanks

Hi @tfjanjua sure, please check out from main, the open release branch is closed for improvement and features

@dhiren-singh-007
Copy link
Contributor

dhiren-singh-007 commented Aug 28, 2024

This api is creating the CompanyInvitation and it make sense to return the Id of CompanyInvitation only.
Because the whole process of creating this invitation is in background process so even if we move the application creation in api itself then one have to still wait for the completion of whole process.
So idea would be to use this invitation id to fetch the Company application Id and Company Id from a different new api (with same set of permissions as Invitation api have).
So far there are two use cases for this

  1. Automation: Automation test need to check the status of company application so it requires application id , which it don't have access in Invitation. So by this change they will be able to get the application id and company id.
  2. CRM Communication : In this case CRM is unable to track the application as there is no reference Id so with this change they can use Invitation Id as reference and then they can call another api to get the application id and company id .
    As this is a background process so in order to get the application id one has to wait for scheduled cron job frequency (currently its 5 min).
    cc @MaximilianHauer @SanaCofinityX @ybidois @tfjanjua

@SanaSalem01 SanaSalem01 changed the title API Create company account | Add "applicationID" in the response body API Create company account | Add invitation id and api for invitation details Aug 28, 2024
@dhiren-singh-007
Copy link
Contributor

UPDATE: As Because of the issue #700 . Company and application creation is moved in api execution itself so now we no longer need an api to retrieve these ids.
So I have updated the @Phil91 PR #960 with this change where Invite api will now return these ids .

@MaximilianHauer
Copy link

@dhiren-singh-007 i assume this solves the ticket ?

@dhiren-singh-007
Copy link
Contributor

@dhiren-singh-007 i assume this solves the ticket ?

Yes @MaximilianHauer , PR #960 will solve this once merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

6 participants