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.

Sign up for GitHub

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

Jump to bottom

implement admin message detail page #1453

Merged
merged 1 commit into from Feb 11, 2023
Merged

implement admin message detail page #1453

merged 1 commit into from Feb 11, 2023

Conversation

notmd
Copy link
Collaborator

@notmd notmd commented Feb 11, 2023 •
edited

  • Add admin message detail page.
  • Add new message tree UI and use/tree endpoint to receive data.
  • New message tree only being used in the admin panel. Not in the user area yet.
  • The current tree UI display is not really well in mobile when the depth > 4. I will address this in the next PR.

Message tree UI
image

notmd
notmd commented Feb 11, 2023
View reviewed changes
website/src/components/Messages/MessageTableEntry.tsx
Comment on lines +50 to +51
avartarPosition?: "middle" | "top";
avartarProps?: AvatarProps;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These props are just to make the new UI compatible with the current UI. Will be cleanup in next PR

@fozziethebeat fozziethebeat merged commit 3f7244c into main Feb 11, 2023
@fozziethebeat fozziethebeat deleted the admin_messages_management branch February 11, 2023 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fozziethebeat fozziethebeat fozziethebeat approved these changes

@k-nearest-neighbor k-nearest-neighbor Awaiting requested review from k-nearest-neighbor

@AbdBarho AbdBarho Awaiting requested review from AbdBarho AbdBarho is a code owner

Assignees
No one assigned
Labels
website
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@notmd @fozziethebeat