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.

Sign up for GitHub

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

Jump to bottom

docs: document the new ui5 tooling task #588

Merged
merged 2 commits into from
Jul 10, 2024
Merged

docs: document the new ui5 tooling task #588

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d2a2229
docs: document the new ui5 tooling task
marianfoo Jul 3, 2024
f4d9c06
docs: update
marianfoo Jul 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 40 additions & 1 deletion docs/pages/GettingStarted.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -399,4 +399,43 @@ builder:
archiveName: uimodule
includeDependencies:
- ui5-cc-spreadsheetimporter
```
```

#### Update `"sap.cloud".service` in Spreadsheet Importer manifest.json

When you are using the decentral deployment and try to deploy, it is possible that the deployment fails with the following error:

`"Service name 'spreadsheetimporter_v1_1_1' and public setting 'true' in embedded manifest.json have to be equal to service name 'xxxxxxx' and public setting 'true' of root manifest.json"`

As of now, according to SAP, this will not be changed.

##### Workaround 1

You can manually update the `service` name in the `manifest.json` of the Spreadsheet Importer to match the service name in the `manifest.json` of your app.

I tried to automate this and you can use the custom task for the UI5 Tooling [`ui5-task-btp-manifest`](https://github.com/spreadsheetimporter/ui5-task-btp-manifest)

###### Install the task

```bash
npm install --save-dev ui5-task-btp-manifest
```

###### Add the task to your `ui5.yaml`

```yml
builder:
customTasks:
- name: ui5-task-btp-manifest
afterTask: replaceVersion
```

This will update the manifest file of the Spreadsheet Importer with the service name of the app like to this path:
`dist/thirdparty/customcontrol/spreadsheetimporter/v1_1_1/manifest.json`

##### Workaround 2

According to the [comment from user TravelTechCode](https://github.com/spreadsheetimporter/ui5-cc-spreadsheetimporter/issues/582#issuecomment-2210564980) you can try to deploy the app without the packaged component first:


> Though one hack i found is that ,if i deploy my app first without the ui5 spreadsheet component and then add and deploy the ui5 spreadsheet component in the second deployment, then it's not causing any deployment error and can be access in the content explorer too.
Loading