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

feat: Enhance createApplicationAccess to allow create applicationAccess by passing existing instance of 'project' to avoid getProject call #2905

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

815are
Copy link
Contributor

@815are 815are commented Feb 11, 2025

There is existing method createApplicationAccess, which calls findAllApps and getProject, where getProject could be performant action for large project.
In this PR enhancing method createApplicationAccess - it should allow to create applicationAccess using previously resolved object of project data and avoid multiple calls for getProject and findAllApps

create application access from project
@815are 815are requested a review from a team as a code owner February 11, 2025 14:32
@815are 815are added the project-access Tickets related to @sap-ux/project-access label Feb 11, 2025
sync function
Copy link

changeset-bot bot commented Feb 11, 2025

🦋 Changeset detected

Latest commit: 0ce6282

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 39 packages
Name Type
@sap-ux/project-access Patch
@sap-ux/abap-deploy-config-sub-generator Patch
@sap-ux/abap-deploy-config-writer Patch
@sap-ux/adp-flp-config-sub-generator Patch
@sap-ux/adp-tooling Patch
@sap-ux/annotation-generator Patch
@sap-ux/app-config-writer Patch
@sap-ux/cap-config-writer Patch
@sap-ux/cards-editor-middleware Patch
@sap-ux/cf-deploy-config-sub-generator Patch
@sap-ux/cf-deploy-config-writer Patch
@sap-ux/create Patch
@sap-ux/environment-check Patch
@sap-ux/fe-fpm-writer Patch
@sap-ux/fiori-annotation-api Patch
@sap-ux/fiori-generator-shared Patch
@sap-ux/flp-config-inquirer Patch
@sap-ux/flp-config-sub-generator Patch
@sap-ux/launch-config Patch
@sap-ux/odata-service-inquirer Patch
@sap-ux/odata-service-writer Patch
@sap-ux/preview-middleware Patch
@sap-ux/project-integrity Patch
@sap-ux/telemetry Patch
@sap-ux/ui5-application-inquirer Patch
@sap-ux/ui5-library-reference-inquirer Patch
@sap-ux/ui5-library-reference-sub-generator Patch
@sap-ux/ui5-library-reference-writer Patch
@sap-ux/ui5-library-writer Patch
@sap-ux/fiori-elements-writer Patch
@sap-ux/fiori-freestyle-writer Patch
@sap-ux/fe-fpm-cli Patch
@sap-ux/abap-deploy-config-inquirer Patch
@sap-ux/deploy-config-generator-shared Patch
@sap-ux/inquirer-common Patch
@sap-ux/ui5-library-sub-generator Patch
@sap-ux/generator-simple-fe Patch
@sap-ux/cf-deploy-config-inquirer Patch
@sap-ux/ui5-library-inquirer Patch

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

changeset
provide project through options instead of creating new function
missed file
changelog
@815are 815are changed the title feat: create application access from project -> createApplicationAccessFromProject feat: Enhance createApplicationAccess to allow create applicationAccess by passing existing instance of 'project' to avoid getProject call Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
project-access Tickets related to @sap-ux/project-access
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant