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.

Sign up for GitHub

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

Jump to bottom

150 documentacion swagger #213

Merged
merged 4 commits into from
Mar 7, 2018
Merged

150 documentacion swagger #213

merged 4 commits into from
Mar 7, 2018

Conversation

poligarcia
Copy link
Contributor

Beni, ya están los comentantarios que me hiciste resueltos.

Podés probar ejecutando:

Cuando se mergee el pr la documentación deberá estar disponible en https://datosgobar.github.io/series-tiempo-ar-api/

@poligarcia
Primer implementación de la documentación swagger en githubpages
97f52d3
@poligarcia
Actualización de deploy de documentación OpenAPI
72c64b1 
* Oculto el explorador de otras especificaciones swagger
* Muevo la documentación a /docs
* Renombro el archivo de la especificación a swagger.yml

refs #150
@poligarcia
Agrego atributo validatorUrl: null para que la UI de swagger no int…
1db3a9d 
…ente validar el yml, ya que la URL es relativa

refs #150
@poligarcia
Elimino objetos del modelo que no son necesarios
1d2d48b 
refs #150
@poligarcia poligarcia requested a review from abenassi March 7, 2018 21:16
@poligarcia poligarcia added this to To do in Servicios - Sprint 6 via automation Mar 7, 2018
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 83.533% when pulling 1d2d48b on 150-documentacion-swagger into 53136f8 on master.

@poligarcia poligarcia merged commit daadb36 into master Mar 7, 2018
Servicios - Sprint 6 automation moved this from To do to Done Mar 7, 2018
@abenassi abenassi added the review label Mar 9, 2018
@lucaslavandeira lucaslavandeira deleted the 150-documentacion-swagger branch March 9, 2018 18:59
@abenassi abenassi removed the review label Jul 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abenassi abenassi Awaiting requested review from abenassi

Assignees
No one assigned
Labels
None yet
Projects
No open projects
Servicios - Sprint 6
  
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@poligarcia @coveralls @abenassi