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

[Finishes #187788139] Control sellers on admin dashboard #33

Merged
merged 1 commit into from
Jul 14, 2024

Conversation

princenzmw
Copy link
Contributor

@princenzmw princenzmw commented Jul 10, 2024

Purpose

The purpose of the PR is to allow the Admin to Control sellers and all other users on the admin dashboard.

Changes Made

Created two pages on the Admin dashboard. One for controlling Sellers, the other for managing all users

Testing Instructions

Navigate to /admin/sellers and /admin/users to see the implementation

Related Issues

Not yet Integrated the real data from the backend

Checklist

Please review the following checklist and make sure all tasks are complete before submitting:

  • Code follows the project's coding standards
  • Changes are covered by tests
  • Documentation is updated (if applicable)
  • All tests pass

@princenzmw princenzmw added the WIP label Jul 10, 2024
Copy link

netlify bot commented Jul 10, 2024

Deploy Preview for e-commerce-mavericks-fn ready!

Name Link
🔨 Latest commit d803952
🔍 Latest deploy log https://app.netlify.com/sites/e-commerce-mavericks-fn/deploys/6693a844eb7b600008f10078
😎 Deploy Preview https://deploy-preview-33--e-commerce-mavericks-fn.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@princenzmw princenzmw force-pushed the 187788139-ft-admin-should-controll-all-sellers branch 6 times, most recently from 8b99c28 to a6e05b4 Compare July 14, 2024 00:42
@princenzmw
Copy link
Contributor Author

Below are the preview images for the pages I worked on:

  1. Preview for the Page where the Admin controls All Users:

Page To control All Users


  1. Preview for the Page where the Admin controls All Sellers:

Page To control All Sellers

Copy link
Contributor

@patrickhag patrickhag left a comment

Choose a reason for hiding this comment

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

👌

@princenzmw princenzmw force-pushed the 187788139-ft-admin-should-controll-all-sellers branch from a6e05b4 to d803952 Compare July 14, 2024 10:28
Copy link
Contributor

@Favor-star Favor-star left a comment

Choose a reason for hiding this comment

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

LGTM

@niyontwali niyontwali merged commit c54df12 into develop Jul 14, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants