Skip to content

Replaced static links to prod UI with docs links #195

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

Merged

Conversation

validbeck
Copy link
Collaborator

Internal Notes for Reviewers

sc-6615

  1. https://app.prod.validmind.ai/ > https://docs.validmind.ai/guide/configuration/register-with-validmind.html
  2. https://app.prod.validmind.ai/model-inventory > https://docs.validmind.ai/guide/model-inventory/working-with-model-inventory.html
  3. In the [Platform UI](https://app.prod.validmind.ai), go to the **Documentation** page for the model you registered earlier. > In the ValidMind Platform UI, go to the **Documentation** page for the model you registered earlier. ([Need more help?](https://docs.validmind.ai/guide/model-documentation/working-with-model-documentation.html))

@validbeck validbeck self-assigned this Sep 23, 2024
@validbeck validbeck added the internal Not to be externalized in the release notes label Sep 23, 2024
@validbeck validbeck assigned cachafla and unassigned cachafla Sep 23, 2024
@validbeck validbeck requested a review from cachafla September 23, 2024 17:47
Copy link
Contributor

PR Summary

This pull request updates various Jupyter notebooks in the project to modify URLs and improve the styling of alert messages. The changes include:

  1. URL Updates: The URLs pointing to the ValidMind platform have been updated to point to the documentation pages instead of the production app. This change affects links for logging in, registering, and accessing various features.
  2. Styling Changes: The color of the text in alert messages has been changed from #222425 to black, and the border color has also been updated to black. Additionally, <br></br> tags have been added for better formatting.

These changes aim to improve the user experience by directing users to the appropriate documentation and enhancing the readability of alert messages.

Test Suggestions

  • Verify that all updated URLs correctly point to the intended documentation pages.
  • Check the formatting of the alert messages in the notebooks to ensure the new styling is applied correctly.
  • Run the notebooks to ensure that there are no broken links or rendering issues.
  • Ensure that the functionality of the notebooks remains unchanged apart from the URL and styling updates.

Copy link
Contributor

@cachafla cachafla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome 👌

@validbeck validbeck merged commit 4e08e51 into main Sep 23, 2024
6 checks passed
@validbeck validbeck deleted the beck/sc-6615/replace-sign-up-now-links-with-the-new-register branch September 23, 2024 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Not to be externalized in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants