Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Use only one api for getting and setting payment instructions. #3913

Closed
aandis opened this issue Feb 5, 2016 · 2 comments
Closed

Use only one api for getting and setting payment instructions. #3913

aandis opened this issue Feb 5, 2016 · 2 comments

Comments

@aandis
Copy link
Contributor

aandis commented Feb 5, 2016

Reticketed from #3909 (comment).

@aandis I think it's best to have one API that both we and third parties use. Maybe we should:

  1. Drop the singular %team/payment-instruction.json.spt
  2. Change our views that use the singular endpoint to use the plural ~/%username/payment-instructions.json.spt instead.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@rohitpaulk
Copy link
Contributor

-ve on this, I think we should have long living endpoints for thirdparties - probably versioned too, like api/v1/username/payment-instructions.json etc.

We could start out with using those internally, but in the future we should be able to change our internal use patterns without changing the thirdparty endpoints.

@aandis
Copy link
Contributor Author

aandis commented Feb 18, 2016

we should be able to change our internal use patterns without changing the thirdparty endpoints.

True. But then the important question to ask is, why was it decided to expose payment-instruction.json as an api instead of using it only internally? Having two apis which do more or less similar stuff can be confusing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants