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

OverwriteHandler flag needs to be at individual handler #165

Closed
stevehu opened this issue Aug 7, 2018 · 2 comments
Closed

OverwriteHandler flag needs to be at individual handler #165

stevehu opened this issue Aug 7, 2018 · 2 comments
Assignees

Comments

@stevehu
Copy link
Contributor

stevehu commented Aug 7, 2018

The current generator only respects OverwriteHandler flag as All or Nothing. We need to change the generator to check if an old handler exists before applying the logic if overwrite it or not. In this case, we can allow new endpoints to be added to the specification and get them generated even OverwriteHandler flag is false.

@stevehu
Copy link
Contributor Author

stevehu commented Aug 7, 2018

Relate to #164

@stevehu stevehu self-assigned this Aug 21, 2018
@stevehu
Copy link
Contributor Author

stevehu commented Aug 22, 2018

The logic has been changed for overwriteHander, overwriteHanderTest and overwriteModel flag from config.json file.

If the flag is false and the file exists, then skip generating the file. With this change, you should set these flags to false most of the time and only set them to true if you want to overwrite existing files.

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

No branches or pull requests

1 participant