Skip to content

[PW-7810] Create templates for small services#202

Merged
antolo-arch merged 3 commits into
developfrom
PW-7810
Jan 24, 2023
Merged

[PW-7810] Create templates for small services#202
antolo-arch merged 3 commits into
developfrom
PW-7810

Conversation

@antolo-arch

Copy link
Copy Markdown
Contributor

Description
Templates for smaller services

Tested scenarios
Tested locally, generated services will be tested in different PRs

Comment thread Makefile
openapi-generator-cli:=java -jar build/openapi-generator-cli.jar
services:=balancePlatform binlookup checkout dataProtection legalEntityManagement management payments payouts platformsAccount platformsFund platformsHostedOnboardingPage platformsNotificationConfiguration recurring storedValue terminalManagement transfer
services:=balancePlatform checkout legalEntityManagement management payments payouts platformsAccount platformsFund platformsHostedOnboardingPage platformsNotificationConfiguration terminalManagement transfer
smallServices:= recurring binlookup dataProtection storedValue balanceControlService

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

could be alphabetic order 🤷

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

😆

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will fix that in the next PR!

Comment thread templates/api.mustache
@@ -1,4 +1,3 @@
{{>partial_header}}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is the partial header removed? Does the api-small class extend the api.mustache class or smth?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

not to include the header in all the classes as it's generated for each subclass, small one is generated once, thus the header stays there

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ahh okay, so only the init files in the big services have that header?

@antolo-arch antolo-arch merged commit b64d8c1 into develop Jan 24, 2023
@antolo-arch antolo-arch deleted the PW-7810 branch February 1, 2023 15:17
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.

3 participants