apinf / platform

Apinf - Open source API management platform with multi proxy and protocol support
https://apinf.com/
European Union Public License 1.1
74 stars 35 forks source link

Layout enhancements of API (Settings > Network, Monitoring, Authentication) #3665

Closed matleppa closed 5 years ago

matleppa commented 5 years ago

Closes #3471 Closes #3679

Changes

Developer checklist

This checklist is to be completed by the PR developer:

Reviewer checklist

Reviewed by: @username1

This list is to be completed by the pull request reviewer:

matleppa commented 5 years ago

The content of the comment is unclear.

Nazarah commented 5 years ago

New wireframes for the UI:

style2

UI now shows clear relationships on how APIs are being called with the proxy host. Unless both unique proxy base path and API base path are added as input, the connection is not visible to API owner.

style3

alert messages and rules to construct proxy/api base path appears below two view fields.

style 4 When Unique and syntactically correct proxy base path and api base path are added to input field, their relation becomes visible with green arrows and texts. Also the formed proxy hosts and api host addresses appear in disabled fields.

matleppa commented 5 years ago

I'd rather not to combine Proxy name with Proxy URL, but keep the Proxy URL + base path combination more mappable with API URL + base path combination. And it would be good not to have redundancy on those. Also I think that the error messages should be next to the field they are referring to.