-
Notifications
You must be signed in to change notification settings - Fork 50
fix: Project type appears twice under system and password prompts when cloud system is selected. #3706
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
Conversation
…dditionalMessage to be consistent with the password prompt mutation.
🦋 Changeset detectedLatest commit: acf883b The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
Changeset OK
Changes look good
Excellent test coverage
Did not test manually
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.
Changes look good
Changeset is good
Test coverage is good
Did not test manually
|
…chart_on_list_v4 * origin/main: chore: apply latest changesets feat(fpm-writer): table building block custom columns (#3665) chore: apply latest changesets fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706) chore: apply latest changesets fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times. (#3683) chore: apply latest changesets update axios version (#3732) chore: apply latest changesets fix(fiori-annotation-api): wrong origin range (#3729) chore: apply latest changesets TBI - Replace service key (UAA/OAuth2) based connections with re-entrance tickets (#3693) chore: apply latest changesets fix: smartlinks target prompt (#3643) chore: apply latest changesets feat: Add ADP Generator Cloud Foundry prompting code (#3518)
…r_RAG_code_gen * origin/main: (26 commits) chore: apply latest changesets feat(odata-service-inquirer): 35947 default table type (#3725) chore: apply latest changesets Fix/35934/incorrect entity list rich text editor (#3724) fix: Remove CF flow from the ADP Generator and fix template resolution problem (#3735) chore: apply latest changesets fix: vocabulary update (#3733) chore: apply latest changesets feat(fiori-mcp-server): add fiori_development_portal_extension.md, etc... (#3717) fix: schedule ui integration tests on every workday (#3620) chore: apply latest changesets feat(fpm-writer): table building block custom columns (#3665) chore: apply latest changesets fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706) chore: apply latest changesets fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times. (#3683) chore: apply latest changesets update axios version (#3732) chore: apply latest changesets fix(fiori-annotation-api): wrong origin range (#3729) ...
* origin/main: (286 commits) chore: apply latest changesets feat(odata-service-inquirer): 35947 default table type (#3725) chore: apply latest changesets Fix/35934/incorrect entity list rich text editor (#3724) fix: Remove CF flow from the ADP Generator and fix template resolution problem (#3735) chore: apply latest changesets fix: vocabulary update (#3733) chore: apply latest changesets feat(fiori-mcp-server): add fiori_development_portal_extension.md, etc... (#3717) fix: schedule ui integration tests on every workday (#3620) chore: apply latest changesets feat(fpm-writer): table building block custom columns (#3665) chore: apply latest changesets fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706) chore: apply latest changesets fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times. (#3683) chore: apply latest changesets update axios version (#3732) chore: apply latest changesets fix(fiori-annotation-api): wrong origin range (#3729) ...
* main: (61 commits) chore: apply latest changesets feat(odata-service-inquirer): 35947 default table type (#3725) chore: apply latest changesets Fix/35934/incorrect entity list rich text editor (#3724) fix: Remove CF flow from the ADP Generator and fix template resolution problem (#3735) chore: apply latest changesets fix: vocabulary update (#3733) chore: apply latest changesets feat(fiori-mcp-server): add fiori_development_portal_extension.md, etc... (#3717) fix: schedule ui integration tests on every workday (#3620) chore: apply latest changesets feat(fpm-writer): table building block custom columns (#3665) chore: apply latest changesets fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706) chore: apply latest changesets fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times. (#3683) chore: apply latest changesets update axios version (#3732) chore: apply latest changesets fix(fiori-annotation-api): wrong origin range (#3729) ...
…ter/building-block-custom-filter * 'main' of github.com:SAP/open-ux-tools: chore: apply latest changesets feat(fiori-mcp-server): add fiori_development_portal_extension.md, etc... (#3717) fix: schedule ui integration tests on every workday (#3620) chore: apply latest changesets feat(fpm-writer): table building block custom columns (#3665) chore: apply latest changesets fix: Project type appears twice under system and password prompts when cloud system is selected. (#3706) chore: apply latest changesets fix(adp)(vscode): SAP Fiori Launchpad Configuration step appears multiple times. (#3683) chore: apply latest changesets update axios version (#3732) chore: apply latest changesets fix(fiori-annotation-api): wrong origin range (#3729) chore: apply latest changesets TBI - Replace service key (UAA/OAuth2) based connections with re-entrance tickets (#3693) chore: apply latest changesets fix: smartlinks target prompt (#3643)



#3698
Add system additional message as a state property of the config prompter in the ADP generator. For a system which requires authentication in order to determine the project type we need to first authenticate to the system. The system additional message contains the project type needed. The message can be set from the system prompt when the system does not require authentication or later in the password prompt when the login completes. In order to prevent from displaying the message twice we maintain a state pointing to the message in the prompter. Only one prompt - system or password could display that message.
Testing done:
Test scenario:
=> When the login completes the project type appears beneath the system prompt.
Test scenario:
=> The project type appears beneath the password prompt.