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

Updated README link to prismlibrary.com documentation #3112

Merged
merged 2 commits into from
Mar 28, 2024

Conversation

DamianSuess
Copy link
Contributor

@DamianSuess DamianSuess commented Mar 27, 2024

Description of Change

Updated README.md's broken link to the Prism website's documentation.

I apologize for not creating an issue for this first. If need be, I can do so for formality.

Bugs Fixed

  • The link would resolve to a 404 Not Found

API Changes

None

Changed:

  • The ReadMe link previously pointed to https://prismlibrary.com/docs/ which it is now, https://docs.prismlibrary.com/docs/

Behavioral Changes

N/A

PR Checklist

  • Has tests (if omitted, state reason in the description)
  • Rebased on top of master at time of PR
  • Changes adhere to coding standard

@DamianSuess DamianSuess changed the title Updated documentation for prismlibrary.com link Updated README link to prismlibrary.com documentation Mar 27, 2024
Copy link
Member

@dansiegel dansiegel left a comment

Choose a reason for hiding this comment

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

thanks for the PR... good catch... if you could actually update the link as shown that would be great. We will be updating the engine that builds the docs which will eventually mean the /docs part will go away.

README.md Outdated Show resolved Hide resolved
Co-authored-by: Dan Siegel <me@dansiegel.net>
@dansiegel dansiegel merged commit 52c04e2 into PrismLibrary:master Mar 28, 2024
2 checks passed
@DamianSuess DamianSuess deleted the patch-1 branch June 1, 2024 19:23
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