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

[BUG]: Static Checks Developer Video Link is Broken #5578

Open
adhiamboperes opened this issue Nov 18, 2024 · 7 comments · May be fixed by #5579
Open

[BUG]: Static Checks Developer Video Link is Broken #5578

adhiamboperes opened this issue Nov 18, 2024 · 7 comments · May be fixed by #5579
Assignees
Labels
bug End user-perceivable behaviors which are not desirable. Impact: Medium Moderate perceived user impact (non-blocking bugs and general improvements). Work: Low Solution is clear and broken into good-first-issue-sized chunks.

Comments

@adhiamboperes
Copy link
Collaborator

Describe the bug

The link to the explainer videos for static checks to support developers is broken.

Steps To Reproduce

  1. Go to https://github.com/oppia/oppia-android/wiki/Interpreting-CI-Results
  2. Click on the link to the developer video
  3. See that the video is missing

Expected Behavior

Open the Youtube video.

Screenshots/Videos

Screenshot 2024-11-18 at 14 48 16

What device/emulator are you using?

No response

Which Android version is your device/emulator running?

No response

Which version of the Oppia Android app are you using?

No response

Additional Context

No response

@adhiamboperes adhiamboperes added bug End user-perceivable behaviors which are not desirable. triage needed Impact: Medium Moderate perceived user impact (non-blocking bugs and general improvements). Work: Low Solution is clear and broken into good-first-issue-sized chunks. and removed triage needed labels Nov 18, 2024
@adhiamboperes
Copy link
Collaborator Author

adhiamboperes commented Nov 18, 2024

@MohitGupta121, I cross checked the developer channel but the Static checks video is not there. Could you please work with Sean to upload the video so we can fix the Wiki?

@adhiamboperes adhiamboperes changed the title [BUG]: Developer Video Links Broken [BUG]: Static Checks Developer Video Link is Broken Nov 18, 2024
@MohitGupta121
Copy link
Member

MohitGupta121 commented Nov 18, 2024

@adhiamboperes Yes, I know these all videos now moves to Oppia offical channel. I making PR for that. Thanks for filing this issue.

@adhiamboperes
Copy link
Collaborator Author

@MohitGupta121, the Accessibilty video links are working correctly. Only the CI one is missing.

@MohitGupta121 MohitGupta121 linked a pull request Nov 18, 2024 that will close this issue
6 tasks
@seanlip
Copy link
Member

seanlip commented Nov 18, 2024

Is this what you are looking for? https://www.youtube.com/watch?v=HLzHQULZbJE

@MohitGupta121
Copy link
Member

MohitGupta121 commented Nov 18, 2024

@adhiamboperes basically I pinged Sean already when these videos moved from Oppia-Developer to Oppia official channel, basically they now completely deleting devloper channel. That's why now I replaced links with Oppia offical channel to be in safer side. Now this is Developers Playlists where we find all videos

@adhiamboperes
Copy link
Collaborator Author

Is this what you are looking for? https://www.youtube.com/watch?v=HLzHQULZbJE

Yes, thanks @seanlip.

@adhiamboperes
Copy link
Collaborator Author

@adhiamboperes basically I pinged Sean already when these videos moved from Oppia-Developer to Oppia official channel, basically they now completely deleting devloper channel. That's why now I replaced links with Oppia offical channel to be in safer side. Now this is Developers Playlists where we find all videos

Seen, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug End user-perceivable behaviors which are not desirable. Impact: Medium Moderate perceived user impact (non-blocking bugs and general improvements). Work: Low Solution is clear and broken into good-first-issue-sized chunks.
Development

Successfully merging a pull request may close this issue.

3 participants