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

Adding buyer and seller info while creating Classified Delivery orders #114

Open
tareq89 opened this issue Aug 10, 2016 · 3 comments
Open
Assignees

Comments

@tareq89
Copy link
Member

tareq89 commented Aug 10, 2016

Background

While creating an order, If a User select classified delivery as an order type, buyer and seller info field should be available to him. The field will contain the buyer's/seller's name, phone no.

In case of buyer, the delivery address should be his default address.
In case of seller, the pickup address should be his default address.

There is an UserRef field on the PersonInfo class at TaskCat, not sure what value should populate this field.

Need review on this before start implementing.

@tareq89
Copy link
Member Author

tareq89 commented Aug 10, 2016

@Shaphil vai, @thehoneymad vai, @fahadwajed, @tonmoy71 ?

@tareq89 tareq89 self-assigned this Aug 10, 2016
@thehoneymad
Copy link
Member

I think this is to be discussed when I roll out the full feature

The UserRef field:
The plan is to make an admin just use userref to populate buyer and seller info if and only if he is a user of GO! fetch. I would hopefully ignore all the other fields if you put an userref and just populate the name and everything else needed from the system automatically.

Default addresses for the order:
The plan is to use something like this.

  1. If you create a classified delivery order you're supposed to give a from and to field (duh)
  2. A guy can sell or accept things from different address. Its okay to have the same address now or different anyway, I wouldn't mind and I dont want really a default feature here (may be from frontend), not so much in backend.

@tareq89
Copy link
Member Author

tareq89 commented Aug 10, 2016

Alright, Understood, Thanks

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

2 participants