[improve][fork] Update fork with lastest changes in upstream #11
Conversation
This file contains 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
* fix: incorrect usage of useEffect callback caused intermittent react errors
* feat: rebuild package lock * feat: update paragon * test: fix axios/jest incompatibility
* fix: should not fetch courseTopics for lti provider * refactor: fetch course topics when provider is edx
…#1164) * refactor: refactor code by creating single instance of header instead of 3 * refactor: refactor courseStatus comparsion code --------- Co-authored-by: SundasNoreen <[email protected]>
This commit installs the Xpert chatbot feature from the frontend-lib-learning-assistant repository into the frontend-lib-learning application. This component is rendered by the Course component. The component is only rendered when a few conditions are satisfied.
…Privacy Policy URL to config (#1171) This commit installs version 1.6.0 of @edx/frontend-lib-learning-assistant, which includes a new user disclosure feature. This commit also includes the Privacy Policy URL to the frontend-app-learning config, because the Xpert learning assistant uses this config value in the disclosure.
…I tweaks (#1174) This commit installs version 1.8.0 of @edx/frontend-lib-learning-assistant, which includes a various UI tweaks.
…tionally render toggle elements (#1178) This commit installs version 1.9.3 of @edx/frontend-lib-learning-assistant, which includes a UI tweak to conditionally render the Xpert toggle button and action message (call-to-action) only when the Xpert sidebar is closed.
They are currently using end, but since we are in the course-start-alert file it should be start instead.
…rning-assistant version (#1182) In order to diagnose low usage rates, we're temporarily adding Segment events for rendering the chat and for closing the call-to-action message. This will help us determine whether Xpert is being successfully rendered and whether Xpert is being ignored by tracking whether learners close the call-to-action message at a high rate. At a minimum, we expect to remove the Segment event for rendering the Xpert, because this will be a very noisy Segment event. We plan to leave it in for a few days just to accumulate data. We will evaluate whether to keep the call-to-action dismissal Segment event based on the data. This commit adds the Segment event for rendering the Xpert. This commit also installs version 1.11.1 of frontend-lib-learning-assistant, which introduces a Segment event for for closing the call-to-action message.
* feat: update react & react-dom to v17 * build: update paragon version * refactor: updated edx packages * refactor: updated react-unit-test-utils * build: update lock file * build: update lock file * build: update lock file * build: update lock file * refactor: bumped frontend-lib-learning-assistant version --------- Co-authored-by: mashal-m <[email protected]>
* feat: upgrade react router to v6 * fix: all test cases affected by react router upgrade * refactor: fix navigations * fix: test cases affectewd due to lib-special-exams * refactor: improve code coverage
* feat: Added Courseware Search container popover * chore: Added unit tests for CoursewareSearch and CoursewareSearchToggle * chore: Updated unit test for CourseTabsNavigation * chore: Partial coverage on Courseware Search Hooks * chore: Finished Courseware Search Hooks unit testing * fix: Fixed an overlook that caused a conditional hook * fix: Reduced bounce timeout on scroll/resize to 100ms * chore: Updated snapshots * chore: Moved @testing-library/react-hooks dep to DEV * chore: Minor adjustments on unit tests * chore: Fixed test issue
…1216) Part of openedx/axim-engineering#23 This updates the `@edx/brand` alias to point to the `brand-openedx` package at the `openedx` scope. This does not impact imports because this package is used via an alias.
* feat: Add CoursewareSearchBar component * fix: lint * fix: Clarified component names and i18n description * test: Add more tests * fix: lint * fix: Made props in CoursewareSearchForm optional
* feat: Added CoursewareSearchResults UI component * fix: Added conditional for undefined case instead of null * fix: Updated code to avoid mutation
Co-authored-by: Stanislav Lunyachek <[email protected]>
* feat: Endpoint usage implementation for Courseware Search Co-authored-by: Simon Chen <[email protected]> * chore: Added tests and error case * fix: remove console log * fix: update tests * fix: update tests * fix: update variables --------- Co-authored-by: Simon Chen <[email protected]> Co-authored-by: German <[email protected]>
Welcome to Codecov 🎉Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment. Thanks for integrating Codecov - We've got you covered ☂️ |
bra-i-am
changed the title
bra-i-am
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR updates the fork of this MFE with the lastest changes in upstream
https://edunext.atlassian.net/browse/DS-713