Create the seller model and API

RESOLVED FIXED

Status

P3
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: andy+bugzilla, Assigned: davidbgk)

Tracking

x86
Mac OS X
Points:
---

Details

(Whiteboard: [zippy])

(Reporter)

Description

5 years ago
Create a seller model that takes a bunch of fields. Let's just make it a simple set:
  * name
  * email
  * bank account number
  * bank account sort code
  * address etc

Create a CRUD API to this.
Document.
(Reporter)

Updated

5 years ago
Whiteboard: [bagpuss]
Priority: -- → P3
(Reporter)

Updated

5 years ago
Blocks: 908771
(Reporter)

Updated

5 years ago
Blocks: 905736
No longer blocks: 908771
No longer depends on: 905736
(Reporter)

Comment 1

5 years ago
The bank account stuff is an implementation detail for Bango, we might not do that every time. If we put anything that sounds like "bank account" people at first glance will have these security worries, so let's not do that, besides it makes for a boring form that.

Let's just gather:

* marketplace user id
* paypal id

Those are pretty harmless.
(Assignee)

Updated

5 years ago
Whiteboard: [bagpuss] → [zippy]
(Reporter)

Comment 2

5 years ago
Assigning to David, because it looks like he created a seller API already.
Assignee: nobody → david
(Reporter)

Comment 3

5 years ago
The documentation for this exists and it looks like the code does:

http://zippypayments.readthedocs.org/en/latest/developer.html#sellers
https://github.com/mozilla/zippy/blob/master/lib/sellers.js
(Assignee)

Comment 4

5 years ago
Yes, but it's outdated. I must rewrite the code using promises/restforms and the tests using the new client to be consistent with recent implementations of models.
(Assignee)

Updated

5 years ago
Whiteboard: [zippy] → [zippy] kanbanzilla[Review]
(Assignee)

Comment 5

5 years ago
Done in https://github.com/mozilla/zippy/commit/cea920d8bc79b98cae4d3ec228307409d06dd2c5
and https://github.com/mozilla/zippy/commit/9b8f9ffd7c18636cbeaaa46dac41be99767c9b2d

There is still a discussion about how to split sellers/bank accounts but there will be a new bug for that specifically.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [zippy] kanbanzilla[Review] → [zippy]
You need to log in before you can comment on or make changes to this bug.