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.

Sign up for GitHub

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

Jump to bottom

[BUG]: In Customize this component dialog box the video start and end time is accepting negative values #18204

Open
KolliAnitha opened this issue May 9, 2023 · 1 comment
Open

[BUG]: In Customize this component dialog box the video start and end time is accepting negative values #18204

KolliAnitha opened this issue May 9, 2023 · 1 comment
Labels
bug Label to indicate an issue is a regression Impact: Low -- ONLY DO IF GOOD FIRST ISSUE Postponing for now, since it doesn't affect users much. Work: Low Solution is known and broken into good-first-issue-sized chunks.

Comments

@KolliAnitha
Copy link

Describe the bug

While inserting video using RTE components, in Customize this component dialog box the video start and end time is accepting negative values

Steps To Reproduce

  1. Login to www.oppiatestserver.org
  2. Go to the Creator dashboard- Click on Create Exploration
  3. In the introduction field click on edit icon - select insert video
  4. Enter youtube URL id as EKoI1hYbpA4 and enter start time as -29, end time as -23
  5. Save content and play preview
  6. See that the video keeps loading because of the start time

Expected Behavior

The fields Video start time in seconds and Video end time in seconds should not accept negative values. A warning message should be displayed on entering a negative value.

Screenshots/Videos

Screen.recording.2023-05-09.3.37.00.PM.webm

What device are you using?

Desktop

Operating System

Other

What browsers are you seeing the problem on?

Chrome

Browser version

112.0.5615.134

Additional context

No response
@KolliAnitha KolliAnitha added bug Label to indicate an issue is a regression triage needed labels May 9, 2023
@shricodev shricodev mentioned this issue May 10, 2023
Closed
@kshitij01042002 kshitij01042002 added Impact: Low -- ONLY DO IF GOOD FIRST ISSUE Postponing for now, since it doesn't affect users much. Work: Low Solution is known and broken into good-first-issue-sized chunks. labels May 25, 2023
@Ykumar1415
Copy link
Contributor

i think this issue is solved. it is working fine now in local development server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Label to indicate an issue is a regression Impact: Low -- ONLY DO IF GOOD FIRST ISSUE Postponing for now, since it doesn't affect users much. Work: Low Solution is known and broken into good-first-issue-sized chunks.
Projects
LaCE Quality Team
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
4 participants
@seanlip @kshitij01042002 @Ykumar1415 @KolliAnitha