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

✨ Load and display moderation status in event list #249

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

foysalit
Copy link
Contributor

This PR hydrates repo and record details on the client side for each event and shows the moderation status of the subject of when viewing event stream.

Screenshot 2024-11-29 at 23 08 52

@arcalinea arcalinea temporarily deployed to subject-status-in-event-stream - ozone-staging PR #249 November 29, 2024 23:11 — with Render Destroyed
@@ -102,12 +103,19 @@ function isMessageSubject(
return subject.messageId !== undefined
}

type ModEventType<T> = { event: T } & ToolsOzoneModerationDefs.ModEventView
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unrelated cleanup to get rid of the ts-ignores

@@ -102,12 +103,19 @@ function isMessageSubject(
return subject.messageId !== undefined
}

type ModEventType<T> = { event: T } & ToolsOzoneModerationDefs.ModEventView

function isModEventType<T>(
Copy link
Contributor

Choose a reason for hiding this comment

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

I have a branch on atproto that should help us get rod of all these !

@foysalit foysalit merged commit 3c1fb44 into main Dec 2, 2024
3 checks passed
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.

3 participants