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

Extensibility walkthrough in capire #407

Closed
wants to merge 19 commits into from
Closed

Extensibility walkthrough in capire #407

wants to merge 19 commits into from

Conversation

danjoa
Copy link
Member

@danjoa danjoa commented Oct 4, 2022

We're using this branch from upcoming revision of http://cap.cloud.sap/docs/guides/extensibility/customization guide.

@danjoa danjoa marked this pull request as draft October 4, 2022 10:14
@chgeo chgeo mentioned this pull request Oct 5, 2022
@tim-sh
Copy link
Member

tim-sh commented May 10, 2023

I updated the branch (rebase onto main, update package{,-lock}.json). We might also want to remove ./orders/ext in favor of ./orders-ext and then merge into main. Thus, we could avoid the ext branch becoming outdated. This would also simplify the guide.
@danjoa what do you think?

PS. compared with main, there are some changes to ./orders:

  • .env is missing
  • there's a model change
  • package.json differs

The latter one might be problematic to have in main, because it is tailored to multitenancy (+toggles+ext).

tim-sh and others added 3 commits May 10, 2023 16:46
The latter being exclusively documented in Capire.
Remove ./orders/ext in favor of ./orders-ext
@danjoa danjoa closed this Aug 7, 2023
@danjoa danjoa deleted the ext branch August 7, 2023 12:59
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

3 participants