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

feat: photo verification dashboard #746

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

ivinayakg
Copy link
Contributor

@ivinayakg ivinayakg commented Apr 10, 2024

Date: 19/04/2024

Developer Name: Vinayak Goyal


Issue Ticket Number

#750

Description

This is going to be a sub-issue of Real-Dev-Squad/website-backend#1924, creating a basic UI for the SUPERUSER to interact and perform actions on the photo-verifications objects.

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screenshot 1

https://github.com/Real-Dev-Squad/website-dashboard/assets/73058928/3703745d-e2d7-4ab9-a623-88412ce376ba
PS - ignore the pictures plz 🙂

Test Coverage

Screenshot 1

Additional Notes

Copy link

cloudflare-workers-and-pages bot commented Apr 12, 2024

Deploying dashboard-rds with  Cloudflare Pages  Cloudflare Pages

Latest commit: c94bbad
Status: ✅  Deploy successful!
Preview URL: https://54e95751.dashboard-rds.pages.dev
Branch Preview URL: https://feat-photo-verification-dash.dashboard-rds.pages.dev

View logs

@ivinayakg ivinayakg force-pushed the feat/photo-verification-dashboard branch from 48f28a4 to c6dcafb Compare April 12, 2024 22:36
@ivinayakg ivinayakg changed the title feat: photo verification dashboard [DRAFT] feat: photo verification dashboard Apr 21, 2024
Copy link
Member

@tejaskh3 tejaskh3 left a comment

Choose a reason for hiding this comment

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

These changes are big, it might take a long to review for someone, which can cause delay to PR reivew. Can we divide this into 2 or more PR if possible.

Comment on lines +9 to +20
// const {
// extensionRequestLogs,
// extensionRequestLogsInSentence,
// } = require('../../mock-data/logs');
// const {
// userSunny,
// userRandhir,
// allUsersData,
// superUserForAudiLogs,
// searchedUserForAuditLogs,
// } = require('../../mock-data/users');
// const { usersStatus } = require('../../mock-data/users-status');
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we remove this?

Copy link
Contributor

@pallabez pallabez 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 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.

4 participants