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

4949-Delete vendors if no purchases #5051

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

jorgedjr21
Copy link
Contributor

Resolves #4949

Description

  • Delete vendors if they does not have any purchases only.
  • Add specs to cover this feature

Include anything else we should know about. -->

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Added system and requests specs.
  • Tested runing human essentials locally

Screenshots

image
image
image

@jorgedjr21 jorgedjr21 changed the title 4949 delete vendors if no purchases 4949-Delete vendors if no purchases Mar 1, 2025
@jorgedjr21
Copy link
Contributor Author

@cielf @dorner I think now this one resolves what is remaining for the issue 4949

@cielf
Copy link
Collaborator

cielf commented Mar 4, 2025

Working as expected.

@cielf cielf requested a review from dorner March 4, 2025 21:48
@jorgedjr21
Copy link
Contributor Author

@dorner applied your suggestions. Also made a small refactor on the spec to check the correct button/links for deactivating and deleting the vendors.

@jorgedjr21 jorgedjr21 requested a review from dorner March 8, 2025 13:44
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

Successfully merging this pull request may close these issues.

Inactivating vendors
3 participants