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

Update GettingStarted.md #524

Closed
wants to merge 1 commit into from
Closed

Conversation

uxkjaer
Copy link

@uxkjaer uxkjaer commented Apr 11, 2024

Added the check on the build script. This took me two hours to find.

Added the check on the build script. This took me two hours to find.
@marianfoo
Copy link
Member

Hi Jakob, thanks for opening the PR.
I'm sorry it took you so much time.
i mention it here on the getting started page:
https://docs.spreadsheet-importer.com/pages/GettingStarted/#setup-decentralized-deployment
3. Add --all to your build script in the package.json

"scripts": {
  ...
  "build": "ui5 build --config=ui5.yaml --all --clean-dest --dest dist",
  ...
}

Do you mean this?

@uxkjaer
Copy link
Author

uxkjaer commented Apr 12, 2024 via email

@marianfoo
Copy link
Member

No worries :)

doco

I see you use ausie slang :D

@marianfoo marianfoo closed this Apr 12, 2024
@uxkjaer
Copy link
Author

uxkjaer commented Apr 13, 2024 via email

@marianfoo
Copy link
Member

Till someone pays for my flight 😁
Would be great to be back some time in the future

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.

None yet

2 participants