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

Connects to #265. Jest test runner errors. #272

Merged
merged 9 commits into from
Oct 23, 2023

Conversation

jimmyzhen
Copy link
Contributor

@jimmyzhen jimmyzhen commented Oct 23, 2023

Key Changes:

  • Resolved a variety of errors/warnings associated with the redesigned Data Hub homepage features and the dependencies used by the features (vis-network and vis-data dependencies were added in the process).
  • Implemented minor CSS styling to address UI issues on the redesigned homepage.
  • Transitioned the embedded video background rendering to a modular component.
  • Uninstalled react-particles-js and tsparticles dependencies as they are no longer needed for the redesigned homepage.

…iling on unsupported JS syntax

Also remove JS particle dependencies as they are no longer being used, as well as the Jest config to attempt a workaround on the failing test.
…page and add local Jest test config to ignore ESM modules
…mepage component

Also add condition to make sure iframe container element to be present before setting the tutorial video's height
@jimmyzhen jimmyzhen merged commit b1b3af6 into dev Oct 23, 2023
3 checks passed
@jimmyzhen jimmyzhen deleted the 265_JZ_Homepage_Redesign_Jest_Test branch January 21, 2024 08:34
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.

2 participants