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

Edit history viewer #3322

Merged
merged 4 commits into from
Jun 9, 2023
Merged

Edit history viewer #3322

merged 4 commits into from
Jun 9, 2023

Conversation

someone13574
Copy link
Contributor

@someone13574 someone13574 commented Jun 7, 2023

Followup PR to #3280

Additons:

  • Endpoint for fetching the revision history of a message
  • Card which allows moderators to view the revision history of a message

Note: Not sure if I'm sorting the uuid's correctly, but it seems to work.

Screenshot:
Screenshot from 2023-06-07 16-33-15

@github-actions
Copy link

github-actions bot commented Jun 7, 2023

pre-commit failed.
Please run pre-commit run --all-files locally and commit the changes.
Find more information in the repository's CONTRIBUTING.md

@someone13574 someone13574 requested a review from notmd June 8, 2023 05:22
Copy link
Collaborator

@notmd notmd left a comment

Choose a reason for hiding this comment

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

Thank you! Web change lgtm

@andreaskoepf
Copy link
Collaborator

andreaskoepf commented Jun 9, 2023

I suggest to render ordered by date descending to display more recent revisions first.

Copy link
Collaborator

@andreaskoepf andreaskoepf left a comment

Choose a reason for hiding this comment

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

Backend changes LGTM..

@andreaskoepf andreaskoepf merged commit c0edd79 into LAION-AI:main Jun 9, 2023
@someone13574 someone13574 deleted the edit-history-viewer branch June 9, 2023 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants