-
Notifications
You must be signed in to change notification settings - Fork 31
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
www/admin: Allow content moderation from a single page #1067
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
c7180d4
to
c459947
Compare
c459947
to
f872d08
Compare
Codecov Report
@@ Coverage Diff @@
## master #1067 +/- ##
===================================================
+ Coverage 46.11759% 46.14048% +0.02288%
===================================================
Files 64 64
Lines 4031 4029 -2
Branches 679 679
===================================================
Hits 1859 1859
+ Misses 1973 1971 -2
Partials 199 199
Continue to review full report at Codecov.
|
It is enough to just suspend the user, which will already make all streams effectively suspended, but only terminate the streams instead of marking them as suspended as well. This allows us to potentially revert the suspension more easily, by not changing the state of every stream in the user account.
bbc7d17
to
6c05d42
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
What does this pull request do? Explain your changes. (required)
This is to make it easier to inspect a given running stream and nuking (suspending) the user.
It allows all that to be done from the same stream detail page, by having an embedded player in
there and adding a button to suspend the user. Additionally, page can now be opened with the
playback ID of the stream, making it easier to go from alert to moderation page (alerts can even
link to it directly).
Specific updates (required)
-
Open stream detail page and check player works, check suspending the user works.
Does this pull request close any open issues?
No.
Screenshots (optional):
Checklist: