#628 Update the API documentation to the new UI
Merged by pingou. Opened by pingou.
api_new_ui  into  master

Download 628.patch
no initial comment
  • before: http://ambre.pingoured.fr/public/pagure_old_new_api.png
  • with this change: http://ambre.pingoured.fr/public/pagure_new_api.png

What happens when pagure.__apiversion__ gets to 10.0 ?

Maybe you should instead pass version=__apiversion__.split('.') to the template instead, and here you'd use version[0] and version[1] ?

Good idea yes

New UI looks much better! :clap:

Other than my earlier comment which you fixed, not sure what else to say.

The HTML seems valid, the JS code as well, I guess?

:+1:

Thanks for the review :)

Metadata