feat(course-outline): add unit title link in expanded outline view#3094
Closed
abhalsod-sonata wants to merge 168 commits into
Closed
feat(course-outline): add unit title link in expanded outline view#3094abhalsod-sonata wants to merge 168 commits into
abhalsod-sonata wants to merge 168 commits into
Conversation
In this commit, - set min height for loader container on course-outline page, so that footer looks stick to bottom of the page - set min height for content on release-notes page, so that footer looks stick to bottom of the page when there is not content on the page - add new env variable to enable/disable release-notes page.
Co-authored-by: Faraz32123 <122095701+Faraz32123@users.noreply.github.com>
Co-authored-by: Faraz32123 <122095701+Faraz32123@users.noreply.github.com>
* chore: update browserslist DB (#5) Co-authored-by: Faraz32123 <122095701+Faraz32123@users.noreply.github.com> * chore: update browserslist DB (#8) Co-authored-by: Faraz32123 <122095701+Faraz32123@users.noreply.github.com> * feat: new pages for release notes * feat: add release notes feature with environment-based toggle and admin access control * feat: handle API errors in release notes component * fix: patch timezone bug and improve design elements * fix: tinymce rtl issue and insert image * feat: enhance release notes UI and layout * test: add tests to improve coverage * fix: addressed minor issues and suggestions * fix: update UI to align with Figma design * fix: resolve css linting warnings --------- Co-authored-by: edX requirements bot <49161187+edx-requirements-bot@users.noreply.github.com> Co-authored-by: Faraz32123 <122095701+Faraz32123@users.noreply.github.com> Co-authored-by: Vivek Ambaliya <vjagdishbhaiambaliya-apphelix@2u.com>
- fix border bottom, it was showing 2 lines, 1 dark and 1 light border line. - fix minimum height for the release notes container when there are no posts or the posts are smaller than normal page height and move the statement to the center of the page.
#13) * fix: Right sidebar design and scrolling behavior in Release Notes page * fix: Adding title to anchor tag * fix: handle modal behavior on post saved state --------- Co-authored-by: Pradeep Patro <ppatro-apphelix@2u.com>
Co-authored-by: Faraz32123 <122095701+Faraz32123@users.noreply.github.com>
Co-authored-by: Faraz32123 <122095701+Faraz32123@users.noreply.github.com>
fix: setting variables type from bool to str
saving alert doesn't disappear even if its saved and keep showing loading status
* fix: group all posts under same date * fix: adjust spacing in right sidebar * fix: hide right sidebar when screen size is smaller * fix: image should not overlap with right sidebar * fix: ui changes - use sentence case for Edit post and Delete post. - take date to second line in schedule tooltip message. - change link color to black --------- Co-authored-by: Muhammad Faraz Maqsood <faraz.maqsood@A006-01130.local>
…on_grading_page fix: saving alert doesn't disappear even its saved
… groups and enhance active note styling
…83) Xpert Unit Summary toggle in Pages & Resources doesn't toggle between enable/disable
…85) Xpert Unit Summary toggle in Pages & Resources doesn't toggle between enable/disable
* fix: support "in progress" status for lib upload When uploading a library archive file during the creation of a new library, the code prior to this commit did not properly handle the "In Progress" state, which is when the celery task doing the archive processing is actively running. Note that this is distinct from the "Pending" state, which is when the task is waiting in the queue to be run (which in practice should almost never happen unless there is an operational issue). Since celery tasks run in-process during local development, the task was always finished by the time that the browser made a call to check on the status. The problem only happened on slower sandboxes, where processing truly runs asynchronously and might take a few seconds. Because this case wasn't handled, the frontend would never poll for updates either, so the upload was basically lost as far as the user was concerned. * feat: add temporary message alert in sections settings tab in libraries (openedx#2734) (openedx#2766) - add temporary message alert in sections settings tab in libraries - increase sidebar width to remove `More` option and display all tabs together (cherry picked from commit 3eeca24) --------- Co-authored-by: David Ormsbee <dave@axim.org> Co-authored-by: Navin Karkera <navin@opencraft.com>
Co-authored-by: Faraz32123 <122095701+Faraz32123@users.noreply.github.com>
Remove "Create Zendesk tickets for suspicious attempts" from Proctoring Settings
Co-authored-by: Faraz32123 <122095701+Faraz32123@users.noreply.github.com>
* feat: add components from Outline page * feat: improve component creation flow and data fetching
- Implemented audio description upload and delete features in the video editor. - Created messages for internationalization in the AudioDescriptionWidget component. - Updated VideoSettingsModal to conditionally render the AudioDescriptionWidget based on context. - Enhanced hooks to manage audio description errors and state. - Added Redux actions and reducers for handling audio description requests. - Developed API methods for uploading and deleting audio descriptions. - Introduced file validation utilities for audio description uploads. - Wrote tests for new functionality, including reducers, selectors, and API methods.
* fix(outline): preserve outline context when adding components * fix: update test cases
* feat: Add audio description functionality to video editor - Implemented audio description upload and delete features in the video editor. - Created messages for internationalization in the AudioDescriptionWidget component. - Updated VideoSettingsModal to conditionally render the AudioDescriptionWidget based on context. - Enhanced hooks to manage audio description errors and state. - Added Redux actions and reducers for handling audio description requests. - Developed API methods for uploading and deleting audio descriptions. - Introduced file validation utilities for audio description uploads. - Wrote tests for new functionality, including reducers, selectors, and API methods. * fix: Add EditorContextProvider to VideoSelectorPage modal - Wrap VideoSelector with EditorContextProvider to provide context for video editor components - Fixes "This component needs to be wrapped in <EditorContextProvider>" error when accessing video gallery modal - Ensures VideoSettingsModal and other editor components have access to EditorContext hooks - Aligns VideoSelectorPage context structure with EditorPage for consistency
* fix: implement transcript editor * fix: remove maxHeight style from transcript container --------- Co-authored-by: pganesh-apphelix <pganesh-apphelix@2U.com>
…100) - Added ComponentMenu component to manage actions (copy, duplicate, delete) for course outline units. - Integrated hooks for delete and duplicate functionalities. - Created tests for ComponentMenu to ensure proper rendering and action handling. - Updated styles to accommodate the new component in the unit card layout. - Enhanced messages for internationalization related to component actions.
…in UnitCard (#101) * feat(course-outline): enhance ComponentMenu and UnitCard with rename functionality - Introduced a new inline rename feature for components in UnitCard, allowing users to edit component names directly. - Updated ComponentMenu to manage menu state and actions more effectively, including closing menus when another is opened. - Added tests to verify the new rename functionality and menu behavior. - Enhanced styles for better UI/UX in the component layout. - Added new messages for internationalization related to renaming components. * fix(course-outline): simplify hooks by removing unitId parameter - Updated hooks in AddComponentWidget, ComponentMenu, and UnitCard to eliminate the unitId parameter, streamlining their usage. - Adjusted related components to ensure compatibility with the new hook signatures. - Improved code readability and maintainability by reducing complexity in hook implementations.
* feat: implement send email for release notes publish * fix: rename unsubscribe function and update related tests * fix: Add email notification features for release notes * fix: add proptype for sendemail * fix: simplify unsubscribe functionality by removing token parameter
…tionality - Updated TitleButton to support external and internal links, allowing the title to be clickable. - Added hover and focus styles for linked titles in CardHeader.scss. - Implemented tests to verify link rendering and interaction behavior in UnitCard. - Ensured that clicking the title link does not expand the unit card, while the expand button still functions as intended.
openedx-webhooks
added
the
open-source-contribution
|
Thanks for the pull request, @abhalsod-sonata! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Submit a signed contributor agreement (CLA)
If you've signed an agreement in the past, you may need to re-sign. Once you've signed the CLA, please allow 1 business day for it to be processed. 🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. DetailsWhere can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.