-
Notifications
You must be signed in to change notification settings - Fork 27
chore: updated documentation with working video setup and sso guide #24
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
base: main
Are you sure you want to change the base?
Conversation
Reviewer's GuideThis PR overhauls the project documentation by migrating and consolidating setup instructions into README.md, enriching the launch workflow with video container and SSO configuration steps, updating access URLs, and correcting minor typos. Flow diagram for SSO setup processflowchart TD
A[Login to Tickets] --> B[Enable Admin Mode]
B --> C[Go to Admin > Global Settings]
C --> D[Generate SSO Keys]
D --> E[Use redirect URL: /talk/oauth2/callback/]
E --> F[Enter Talk Container]
F --> G[Run create_social_apps]
G --> H[SSO enabled for Talk]
Flow diagram for video container setup and port configurationflowchart TD
A[Start eventyay-video container] --> B[Import sample config]
B --> C[Create video superuser]
C --> D[Change port in config.js from 8443 to 8375]
D --> E[Access video at /control]
File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
Hey @Saksham-Sirohi - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
670e5f2
to
357b609
Compare
357b609
to
18f3455
Compare
I got the below error when I tried to setup ey-video: Traceback (most recent call last): |
The migrate command has been added now in order to fix this also, for initiating the sample file event loop is closed is expected |
Thank you, I can now access video component now |
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.
Please also change the document which part is necessary for development and which part for actual deployment
5c4f8e8
to
5fc8cbe
Compare
5fc8cbe
to
0163e67
Compare
3caeb4b
to
ebcf17b
Compare
ebcf17b
to
46bb65b
Compare
Closes #21
Summary by Sourcery
Update development README with clarified launch instructions, video container setup, and SSO configuration guide.
Documentation: