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

[Erlang][Server] Add two options: packageName and openAPISpecName #1303

Merged
merged 3 commits into from
Oct 24, 2018
Merged

[Erlang][Server] Add two options: packageName and openAPISpecName #1303

merged 3 commits into from
Oct 24, 2018

Conversation

feihongmeilian
Copy link
Contributor

packageName : Specify the prefix to the erl file that generate by openapi-generator.
openAPISpecName : Specify the json file name generated by yaml file

@wing328
Copy link
Member

wing328 commented Oct 24, 2018

@feihongmeilian the change looks good to me.

cc @tsloughter

@@ -69,6 +69,6 @@ prepare_validator() ->

get_openapi_path() ->
{ok, AppName} = application:get_application(?MODULE),
filename:join({{packageName}}_utils:priv_dir(AppName), "openapi.json").
filename:join({{packageName}}_utils:priv_dir(AppName), "{{openAPISpecName}}.json").
Copy link
Member

Choose a reason for hiding this comment

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

Please use {{{openAPISpecName}}} instead to obtain the original value.

Copy link
Member

@wing328 wing328 left a comment

Choose a reason for hiding this comment

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

LGTM (the Travis CI failure is not related to this PR)

@wing328 wing328 merged commit 24ea884 into OpenAPITools:master Oct 24, 2018
@wing328 wing328 changed the title For erlang-server. add two options: packageName and openAPISpecName [Erlang][Server] Add two options: packageName and openAPISpecName Oct 24, 2018
@wing328
Copy link
Member

wing328 commented Oct 31, 2018

@feihongmeilian thanks for the PR, which is included in the v3.3.2 release: https://twitter.com/oas_generator/status/1057649626101112832

A-Joshi pushed a commit to ihsmarkitoss/openapi-generator that referenced this pull request Feb 27, 2019
…penAPITools#1303)

* Implement the ability to read packageName and openAPISpecName from a json file

* Replace '{{openAPISpecName}}' with '{{{openAPISpecName}}}'

* Update erlang-petstore-server
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.

None yet

2 participants