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

temporarily only include 'settings' #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

InsaneZein
Copy link
Contributor

for https://issues.redhat.com/browse/RHCLOUD-32400

there's an issue right now where the widget doesn't match the actual learning resources page, because I think the page only includes the "settings" bundle. This will temporarily include just the "settings" bundle.

@InsaneZein InsaneZein requested review from karelhala and a team June 24, 2024 21:09
@karelhala
Copy link
Contributor

This is actually a desired outcome. Until we have a proper learning resources center we should show on the widget full list of bookmarked learning resources. If user doesn't have any learning resrouces bookmarked we can send them to settings bundle because this bundle holds learning resources of a couple of other bundles. But if they bookmark anything we should show it on the landing page.

Example is: user opens openshift bundle and bookmarks openshift learning resrouce. They later open landing page and see imediately bookmarked learning resource from openshift bundle.

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