-
Notifications
You must be signed in to change notification settings - Fork 6
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: add get started & Who is Behind sections landing page anonymous users (#2891) #2896
feat: add get started & Who is Behind sections landing page anonymous users (#2891) #2896
Conversation
You can access the deployment of this PR at https://renku-ci-ui-2896.dev.renku.ch |
933f59b
to
8c82e8c
Compare
8c82e8c
to
9e61ab3
Compare
9e61ab3
to
7b1dd2b
Compare
7b1dd2b
to
30d2661
Compare
30d2661
to
1a6a071
Compare
1a6a071
to
4207369
Compare
4207369
to
e96585a
Compare
e96585a
to
d38263d
Compare
d38263d
to
9e08835
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarding the message in the skeleton card when there is an error loading the project, I added it to quickly identify errors in deployment, such as missing the project path or an invalid path. The purpose is to quickly recognize the mistake and address it.
If the project isn't reachable anymore (E.G. accidentally deleted or otherwise not available) users will see the error.
In general, the UI should only show errors targeting users, not administrators. Those should be logged though Sentry instead.
Not blocking for merging the PR since we are short on time, but it would be nice to have this changed later.
1d00d96
to
cd9d471
Compare
…leton and add back button from landing page
5f0b850
to
ca6cf9c
Compare
ca6cf9c
to
b35394b
Compare
9ee1bfb
into
3357-redesign-landing-page
Tearing down the temporary RenkuLab deplyoment for this PR. |
* feat(client): add hero section landing page anonymous users (#2885) (#2890) * feat(client): add get started & Who is Behind sections landing page anonymous users (#2891) (#2896) * feat(client): Add Teaching & What Is Renku sections landing page (#2911)(#2912) (#2913) * fix(client): Add project path and dataset slug home parameters for the landing page Fix #2895 #2891 #2897 #2911 #2912
PR to add new getStarted and Who is Behind sections for landing page anonymous users.
Testing
To include or modify a project for testing the start session on the landing page, configure the HOMEPAGE_PROJECT_PATH in the client/run-telepresence.sh script.
No included here:
Fix #2891
Fix #2897
/deploy renku=2891-get-started-section extra-values=global.renku.cli_version=2.7.0 #persist