Merged
Conversation
sammy-sandhu
added a commit
that referenced
this pull request
Aug 19, 2025
* adding accordion for derived courses * course id page * updated workflow * Modified workflow/added Sonarcloud config * Removed test branch * Derived courses initial UI setup * spacing fixed * took padding for accordion to match wireframe added showMore flag * show section only if derived courses data exists * show more button and implementation * Update [courseId].js made changes to show/hide courses button to match wireframe design * Removed dev branch from github actions (#112) * Fixing Lint for Ironbank * Update cd-workflows.yml * XDS-UI Env Var Github Actions (#116) * adding env for backend connection test * Removed .env * adding backend secret to workflow for backend connection test * adding backend secret to workflow for backend connection test * adding backend secret to workflow for backend connection test * environment variables actions testing * environment variables actions testing * Cleaning up workflow/Dockerfile * Cleaning up workflow/Dockerfile * notifications * rerender on mark all as read click * fix spelling * Update cd-workflows.yml * Update Dockerfile to trigger workflow * Update cd-workflows.yml * reduce width on derived courses have derived courses link to that course page * change timestamp format * adding platform to xAPI * Correction platform in xAPI * Edited platform * icon css to position horizontal * updating the hidden derived courses to a clickable link * endpoints updated * Mark all read button functionality update * adding a check for no filters * Update cd-workflows.yml updating actions workflow trigger on dev branch * Update README.md * logo dynamic with config image (#120) * updates to course information mapping * node version update * Updated ECR tag * adding check * Use xAPI Forwarding (#131) * drop in new xapi code and tests alongside old * feed course url to share btn * instrument share button with shared learning resource * remove xapi instrumentation from view button * remove xapi instrumentation from course spotlight card * remove xapi instrumentation from searchresult card * instrument create saved search modal with saved search statement * added new instrumentation to save list modal * added new instrumentation to saved modal course page * added explored instrumentation to course page * expanded test for course page * fix index test * added search instrumentation to index page * remove instrumentation from list page * fix one test in search * migrated search to new instrumentation * rip out old instrumentation * fix header test * fixed default layout test * fix final failing test why not * refactor xAPi mock * mock specific xapi events and migrate tests * further refactor * test expectation of saved * add negative case too * update test for index search * Latest code sync with P1 master (#133) * Latest code sync with P1 master * Adding dev branch for workflow PR run --------- Co-authored-by: Simeranjeet Sandhu <simeransandhu@deloitte.com> * Fixed Lint --------- Co-authored-by: schittlur <schittlur@deloitte.com> Co-authored-by: Simeranjeet Sandhu <simeransandhu@deloitte.com> Co-authored-by: RyanKingDeloitte <144351041+RyanKingDeloitte@users.noreply.github.com> Co-authored-by: schittlur <78377530+schittlur@users.noreply.github.com> Co-authored-by: Luis Reyes Madrid <lreyesmadrid@deloitte.com> Co-authored-by: JDTobin <90791836+JDTobin@users.noreply.github.com> Co-authored-by: kjijo <kjijo@deloitte.com> Co-authored-by: Milton Reder <milton.reder@gmail.com>
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
2 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.