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

fix: generated app does not start when choosing JSON schema #3034

Merged
merged 1 commit into from
Jan 31, 2023
Merged

fix: generated app does not start when choosing JSON schema #3034

merged 1 commit into from
Jan 31, 2023

Conversation

w3cj
Copy link
Contributor

@w3cj w3cj commented Jan 31, 2023

Summary

  • Tell us about the problem your pull request is solving.
    • After generating an app with "JSON schema" definition format
    • Error Property '$id' is missing in type is thrown for the file src/configuration.ts
    • Error Cannot redeclare block-scoped variable 'userDataSchema'. is thrown for src/services/users/users.schema.ts
  • Are there any open issues that are related to this?
    • No
  • Is this PR dependent on PRs in other repos?
    • No

@netlify
Copy link

netlify bot commented Jan 31, 2023

‼️ Deploy request for feathers-dove rejected.

Name Link
🔨 Latest commit c33207f

@daffl daffl merged commit 7b8250b into feathersjs:dove Jan 31, 2023
@daffl
Copy link
Member

daffl commented Jan 31, 2023

Oof, sorry about that! Once again learned the lesson that if it isn't covered with tests it will break at some point 😆 I now added full JSON schema tests to the CI test matrix in #3035 and with your fix they all pass now, thank you!

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.

2 participants