Skip to content

Added open api support#1772

Open
Ben-Edwards-cgi wants to merge 35 commits intomasterfrom
added-open-api-support
Open

Added open api support#1772
Ben-Edwards-cgi wants to merge 35 commits intomasterfrom
added-open-api-support

Conversation

@Ben-Edwards-cgi
Copy link
Copy Markdown
Contributor

Added open api support

# Conflicts:
#	.gitignore
#	package.json
#	yarn.lock
# Conflicts:
#	.gitignore
#	package.json
#	tsconfig.json
@Ben-Edwards-cgi Ben-Edwards-cgi removed their assignment Dec 2, 2025
…s service

- Deleted multiple auto-generated interface files related to individual and organisation details, aliases, payment terms, and creditor accounts.
- Updated tsconfig.json to exclude generated interfaces and types from compilation.
- Cleaned up tsconfig.spec.json to exclude generated files from tests.
@iamfrankiemoran
Copy link
Copy Markdown
Contributor

Clarified the OpenAPI workflow: yarn generate:openapi now produces reference-only models under interfaces/generatedand types/generated, gitignored and excluded from TS/tests. Developers can run the command to fetch/merge the BE specs quickly and copy/paste shapes into our hand-written interfaces without importing generated files in app code. README and summary docs updated to make this explicit.

Copy link
Copy Markdown
Contributor

@Arnabsubedi233 Arnabsubedi233 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@louisbriggs louisbriggs left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@max-holland max-holland left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants