Skip to content

Feat/#tc 30 update order status#274

Merged
Kwandes merged 17 commits intoadmin-pagefrom
feat/#TC-30-update-order-status
Dec 7, 2021
Merged

Feat/#tc 30 update order status#274
Kwandes merged 17 commits intoadmin-pagefrom
feat/#TC-30-update-order-status

Conversation

@CristiPV
Copy link
Contributor

@CristiPV CristiPV commented Dec 3, 2021

This pull request closes:

✔️ Closes: #197
🏁 Jira Issue: TC-30

Which service(s) does this issue affect:

  • 🛰️ API
  • 🛒 webstore
  • 🎓 admin-page

Types of changes

  • 🐛 Bug fix (a non-breaking change which fixes an issue)
  • 🧱 New feature (a non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🤖 CI/CD (a change to the CI/CD pipeline)
  • ⚗️ Refactoring (an update to some already existing code)

Documentation Updates

  • 📂 No need for updates
  • 📁 Requires an update ( not done within this issue )
  • 🗃️ Has been updated ( done within this issue )

Testing checklist

  • 💪 Manual tests
  • 🔧 Automatic tests

Browser compatibility - [Frontend only]

  • Tested on Safari
  • Tested on Chrome
  • Tested on Mozilla Firefox

What changes have been done within this issue?

I turned the order status label into an input of type select which contains a list of all statuses.
I created a new API endpoint for updating the order status.
When the status of an order is changed, the API is called to be updated.

How should this be manually tested?

By going to the orders page and updating the status of an order.

Screenshots or example usage

  • Go to order page.
  • Choose an order from the table.
  • Click on the status label of the chosen order.
  • Select a new status from the list.

@CristiPV CristiPV requested a review from Kwandes December 3, 2021 08:46
@DryDish DryDish assigned DryDish and unassigned DryDish Dec 3, 2021
@CristiPV CristiPV added this to the Post-release (Milestone 5) milestone Dec 3, 2021
@CristiPV CristiPV added the feat 🎸 A new feature label Dec 3, 2021
@CristiPV CristiPV self-assigned this Dec 3, 2021
Copy link
Member

@Kwandes Kwandes left a comment

Choose a reason for hiding this comment

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

Requested a few changes

You have formatted a few files with your formatted which is ignoring the formatting rules for the API. I suggest that you ignore that and simply let me know when you're finished with the changes and I will format the API back to the convention.

@Kwandes Kwandes merged commit 927eee1 into admin-page Dec 7, 2021
@Kwandes Kwandes deleted the feat/#TC-30-update-order-status branch December 7, 2021 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 🎸 A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants