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

feat: adds filter by tags and contentType to library home [FC-0059] #50

Conversation

yusuf-musleh
Copy link

No description provided.

yusuf-musleh and others added 30 commits June 3, 2024 15:44
When lib mode is set to "mixed", both "Libraries" and "Legacy Libraries" tabs are show in the Studio Home. When "Libraries" is clicked, v2 libraries are fetched, when "Legacy Libraries" is clicked, v1 libraries are fetched.

When lib mode is set to "v1 only" or "v2 only", only one tab "Libraries" is show and only the respective libraries are fetched when the tab is clicked.
This is to switch between different library modes.
The path updates when selecting tabs, when accessing the url with the path directly it will open its respective tab. Navigating using the browser back/forward buttons is also supported.
This commit is temporary as the current frontend build system in tests doesnt support TS syntax. That should be fixed soon, and this commit should be removed.
This is a temporary commit since there are currently no webpack loaders that support tsx files in the test running. This commit should be removed once that is fixed upstream.
When lib mode is set to "mixed", both "Libraries" and "Legacy Libraries" tabs are show in the Studio Home. When "Libraries" is clicked, v2 libraries are fetched, when "Legacy Libraries" is clicked, v1 libraries are fetched.

When lib mode is set to "v1 only" or "v2 only", only one tab "Libraries" is show and only the respective libraries are fetched when the tab is clicked.
This is to switch between different library modes.
The path updates when selecting tabs, when accessing the url with the path directly it will open its respective tab. Navigating using the browser back/forward buttons is also supported.
This commit is temporary as the current frontend build system in tests doesnt support TS syntax. That should be fixed soon, and this commit should be removed.
This is a temporary commit since there are currently no webpack loaders that support tsx files in the test running. This commit should be removed once that is fixed upstream.
@yusuf-musleh yusuf-musleh force-pushed the yusuf-musleh/temp-fix-types-to-deplpoy-library-v2-filters branch from dc090ca to 4f46250 Compare July 9, 2024 10:31
Copy link

codecov bot commented Jul 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (taxonomy-sandbox-20240709@83489b0). Learn more about missing BASE report.

Additional details and impacted files
@@                     Coverage Diff                      @@
##             taxonomy-sandbox-20240709      #50   +/-   ##
============================================================
  Coverage                             ?   92.68%           
============================================================
  Files                                ?      693           
  Lines                                ?    12339           
  Branches                             ?     2661           
============================================================
  Hits                                 ?    11436           
  Misses                               ?      872           
  Partials                             ?       31           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yusuf-musleh yusuf-musleh marked this pull request as ready for review July 9, 2024 10:40
@yusuf-musleh yusuf-musleh merged commit 4f46250 into taxonomy-sandbox-20240709 Jul 9, 2024
5 checks passed
@bradenmacdonald bradenmacdonald deleted the yusuf-musleh/temp-fix-types-to-deplpoy-library-v2-filters branch September 26, 2024 16:38
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.

4 participants