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

Clarify semantics #1344

Closed
philippeluickx opened this issue Jul 26, 2016 · 6 comments
Closed

Clarify semantics #1344

philippeluickx opened this issue Jul 26, 2016 · 6 comments
Assignees
Milestone

Comments

@philippeluickx
Copy link
Contributor

philippeluickx commented Jul 26, 2016

To me it is not clear what a frontend and backend prefix is. This is being used in the info display for an API, e.g. https://apika.digipalvelutehdas.fi/api/A844qMfhmoAFGAJMu

I googled the terms and only get meaningful results on our own project.

@bajiat
Copy link
Contributor

bajiat commented Sep 7, 2016

@brylie Doesn't your PR #1457 already handle this? Or are there other instances of frontend and backend prefixes outside the Proxy tab in Profile view.

@brylie
Copy link
Contributor

brylie commented Sep 7, 2016

@bajiat it is definately related. @Nazarah and I decided to change frontend and backend for clarity:

  • frontend -> Proxy
  • backend -> API

We also changed the word 'prefix' to 'base path', to match the Open API Specification.

The base path on which the API is served, which is relative to the host.

The changes can be seen, as you mentioned, in the Proxy Settings UI made available in PR #1457:

proxy settings form UI

@bajiat
Copy link
Contributor

bajiat commented Sep 7, 2016

So should we close the issue once PR is merged?

@brylie
Copy link
Contributor

brylie commented Sep 7, 2016

I can associate this issue with the PR, if you agree that it is closed when the PR is merged. There may be a related documentation task, where we create a diagram to clarify these concepts. E.g.

Apigee API proxy diagram

@bajiat
Copy link
Contributor

bajiat commented Sep 7, 2016

Documentation could be a non-urgent continuation task.

@bajiat bajiat added in progress and removed backlog labels Sep 7, 2016
@brylie
Copy link
Contributor

brylie commented Sep 7, 2016

OK, I have added a 'closes' reference to this task in the PR.

@brylie brylie added this to the Sprint 30 milestone Sep 7, 2016
@brylie brylie self-assigned this Sep 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants