Skip to content

Conversation

@jyaistMap
Copy link
Collaborator

@nparavicini7

  • submitted this PR with some potential changes

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@@ -1,337 +1,347 @@
{
Copy link
Collaborator

@nparavicini7 nparavicini7 Jul 9, 2025

Choose a reason for hiding this comment

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

The hyperlink for ContentManager GIS is wonky


Reply via ReviewNB

@nparavicini7
Copy link
Collaborator

@jyaistMap unfortunately nbreview did the thing where it counts the entire notebook as new instead of just your changes, so I can't really tell what specifically you did. However, reading over it all, it looks good, minus the one hyperlink with the weird formatting. Were there any notable changes you made or just random little words here and there?

@jyaistMap
Copy link
Collaborator Author

hey @nparavicini7 - I can't figure out any kind of pattern as to when ReviewNB does that annoying thing. I just replaced/rephrased things here and there - and added hyperlinks with the relative formatting that is confusing to explain. Trust me and I'll verify on the preview sites.

@nparavicini7 nparavicini7 merged commit 66113f2 into ocm_guide Jul 9, 2025
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.

3 participants