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

fix: Accessibility issue on courseDashboard #327

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

omerhabib26
Copy link
Contributor

Description:

  • added content description for views
  • make list as default view for MainDashboard

fix: LEARNER-10021

@volodymyr-chekyrta
Copy link
Contributor

Hi @omerhabib26, the Primary Course experience is the default setting for the Open edX application.
We cannot change it.

@omerhabib26
Copy link
Contributor Author

Hi @omerhabib26, the Primary Course experience is the default setting for the Open edX application. We cannot change it.

As per my understanding the primary course experience API is on sandbox, that's why I changed it to the list. If the change is now available on prod, I can revert it?

@volodymyr-chekyrta
Copy link
Contributor

Hi @omerhabib26, the Primary Course experience is the default setting for the Open edX application. We cannot change it.

As per my understanding the primary course experience API is on sandbox, that's why I changed it to the list. If the change is now available on prod, I can revert it?

Changes are not available on prod, so we use the develop branch.
Since the changes will be on prod, we merge this feature into master.

- added content description for views
fix: LEARNER-10021
@omerhabib26 omerhabib26 force-pushed the omer/LEARNER-10021 branch from ec790f7 to bbc981d Compare June 3, 2024 14:52
Copy link
Contributor

@volodymyr-chekyrta volodymyr-chekyrta left a comment

Choose a reason for hiding this comment

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

@omerhabib26 thank you!
👍

@k1rill k1rill merged commit 22ee176 into openedx:develop Jun 6, 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