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: How to add a third party module #36

Merged

Conversation

longieirl
Copy link
Contributor

  • Append a new sample project showing the introduction of a third party module i.e. xml-js

@longieirl longieirl self-assigned this Dec 14, 2023
@longieirl
Copy link
Contributor Author

@ashleytung can you please review, thanks.

@longieirl longieirl changed the title feat: Add third party sample project feat: How to add a third party module Jan 4, 2024
Copy link
Contributor

@ashleytung ashleytung left a comment

Choose a reason for hiding this comment

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

Hi John, i reviewed the read me files and added a a few comments on branding names.

- Sub-account destination `northwind` is available, please see attached `northind` configuration

# Sample Project
A [Fiori tools Freestyle UI application](./ztravelapp/README.md) with an external library `xml-js` enabled.
Copy link
Contributor

Choose a reason for hiding this comment

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

freestyle app should be referred to as freestyle SAPUI5 app. Also, I suggest to make the sentence a bit more descriptive. Example: "This freestyle SAPUI5 application demonstrates how to add an external library xml-js to the project"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@@ -0,0 +1,66 @@
# Adding External Library to your Fiori UI application
Copy link
Contributor

Choose a reason for hiding this comment

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

SAP Fiori instead of Fiori

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done


From your SAP BTP cockpit, select Instances and Subscriptions, select SAP Business Application Studio which will open a new tab into your dev space manager. Generate a SAP Fiori dev space.

# Generate a Fiori UI Freestyle Application
Copy link
Contributor

Choose a reason for hiding this comment

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

"freestyle SAPUI5 application"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

longieirl and others added 2 commits January 5, 2024 09:26
changed from "Fiori" to "SAP Fiori"
Copy link
Contributor

@ashleytung ashleytung 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. Thank you.

@longieirl
Copy link
Contributor Author

looks good. Thank you.

Can you please merge as I dont have access? Thanks.

@theStiggySpd theStiggySpd merged commit d972c78 into SAP-samples:main Jan 11, 2024
1 check passed
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.

4 participants