Skip to content
This repository has been archived by the owner on May 21, 2020. It is now read-only.

add orders history to my account #213

Merged
merged 27 commits into from
Mar 18, 2020
Merged

add orders history to my account #213

merged 27 commits into from
Mar 18, 2020

Conversation

Qrzy
Copy link
Collaborator

@Qrzy Qrzy commented Feb 24, 2020

No description provided.

@Qrzy Qrzy linked an issue Feb 24, 2020 that may be closed by this pull request
@Qrzy Qrzy changed the title add orders to useUser composable add orders history to my qccount Feb 25, 2020
@Qrzy Qrzy changed the title add orders history to my qccount add orders history to my account Feb 25, 2020
@Qrzy Qrzy force-pushed the my-orders-api-client branch 3 times, most recently from 313fd32 to 5bf40c7 Compare February 26, 2020 07:34
@Qrzy Qrzy marked this pull request as ready for review February 26, 2020 07:35
@Qrzy Qrzy force-pushed the my-orders-api-client branch 2 times, most recently from 53fb3d4 to f9e19e0 Compare February 26, 2020 07:43
Copy link
Collaborator

@andrzejewsky andrzejewsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@filrak do we need a another field for orders or can we include it in the user field? What do you think? I think having different field just for orders is even better (no enhance functions in the future etc.)

@Qrzy Qrzy force-pushed the my-orders-api-client branch 6 times, most recently from 3bf122c to e3427d2 Compare February 28, 2020 07:19
@Qrzy Qrzy force-pushed the my-orders-api-client branch 5 times, most recently from 149c5c1 to 44cd785 Compare February 28, 2020 09:43
@Qrzy Qrzy modified the milestone: CT integration Mar 13, 2020
@Qrzy Qrzy removed the blocked label Mar 13, 2020
@CLAassistant
Copy link

CLAassistant commented Mar 16, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@patzick patzick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just some minor things here

@andrzejewsky andrzejewsky self-requested a review March 18, 2020 09:43
@andrzejewsky andrzejewsky merged commit 32aadd1 into master Mar 18, 2020
@andrzejewsky andrzejewsky deleted the my-orders-api-client branch March 18, 2020 10:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants