Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

[frontend] fix #174

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

UGisBusy
Copy link
Collaborator

@UGisBusy UGisBusy commented Jan 10, 2024

Ex. Resolves #173

Changes

  • fix the issue (the problem mention in the issue)

Testing Approach (if unconventional)

docker compose up --build
  • seller page mayhem
    • when the problem is associate with shop being desable, it will alert message about that and return to last page
    • when the problem is not associate with shop being desable, it will alert msg but leave you there
    • when u "did the main function" in that page, itll return to last page
      • edit successfully on edit product/coupon
      • delete successfully on edit product/coupon
      • add successfully on add product/coupon
  • credit card
    • when u r click pay on checkout, if u have no credit card, itll alert "pls add new card"
    • when u r click pay on checkout, if u have any credit card, itll alert "pls select a card"
    • successfuly pay if pass those check

@UGisBusy UGisBusy self-assigned this Jan 10, 2024
Copy link
Collaborator

@NOOBDY NOOBDY left a comment

Choose a reason for hiding this comment

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

LGTM

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

Successfully merging this pull request may close these issues.

ohno, frontend fuck up
2 participants