Skip to content

Harmonized openapi-gen.json, added to package.json and added fixup script#1132

Merged
dbeuchler merged 4 commits intodevfrom
feature/ngopenapigen-dev-dependency
Mar 3, 2026
Merged

Harmonized openapi-gen.json, added to package.json and added fixup script#1132
dbeuchler merged 4 commits intodevfrom
feature/ngopenapigen-dev-dependency

Conversation

@dbeuchler
Copy link
Member

@dbeuchler dbeuchler commented Feb 27, 2026

  • Every web project uses the same file-name for api-definition
  • Added ng-openapi-gen as dev-dependency to use defined version on each generation
  • Node.js does not trust self-signed certificates by default (which localhost typically uses for HTTPS)
    • Workaround by NODE_TLS_REJECT_UNAUTHORIZED=0
  • Added script to replace entry files with ngx-web-framework exports and adding copyright header to generated files

I regenerated all apis with the new script. The location of the script is currently in ./src folder because I have no better location for it. Maybe we add it to the web-framework later if it is interesting in other projects too.

There are tons of changes to the last empty line in the generated files. This comes from new ng-openapi-gen version. For review, do it commit by commit.

@dbeuchler dbeuchler added this to the Framework 10.x milestone Feb 27, 2026
@dbeuchler dbeuchler self-assigned this Feb 27, 2026
@dbeuchler dbeuchler added the enhancement New feature or request label Feb 27, 2026
@dbeuchler dbeuchler force-pushed the feature/ngopenapigen-dev-dependency branch from bc35516 to 58fe018 Compare February 27, 2026 11:49
@dbeuchler dbeuchler changed the title Harmonized openapi-gen.jso, added to package and fixup script Harmonized openapi-gen.json, added to package.json and added fixup script Feb 27, 2026
@dbeuchler dbeuchler force-pushed the feature/ngopenapigen-dev-dependency branch from 58fe018 to 1d64b05 Compare March 2, 2026 13:14
Copy link
Member

@1nf0rmagician 1nf0rmagician left a comment

Choose a reason for hiding this comment

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

Nice! I like that all services are up to date again. We had apparently missed some stuff there in the past.

Not sure when the script will be "worth it" compared to the time spend exchanging the references. But since you have already invested the time it will be in any case now 😁

@dbeuchler dbeuchler merged commit f13f3dc into dev Mar 3, 2026
14 of 15 checks passed
@dbeuchler dbeuchler deleted the feature/ngopenapigen-dev-dependency branch March 3, 2026 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants