-
Notifications
You must be signed in to change notification settings - Fork 515
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
Deploy on BTP #793
base: main
Are you sure you want to change the base?
Deploy on BTP #793
Conversation
…oyment config files
orders/package.json
Outdated
"[production]": { | ||
"auth": "xsuaa", | ||
"db": "hana", | ||
"messaging": "enterprise-messaging" | ||
} |
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.
Need to test if this really is necessary - these configurations should already be the default for production
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.
"messaging": "enterprise-messaging" still needed
@@ -0,0 +1,139 @@ | |||
{ |
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.
Need to test if this is already generated during build if not present - if not, would be good to improve the tooling
d6c5c62
to
ff9224f
Compare
minimal changes required to deploy bookshop in sharedDB mode: