8000 Support updates by kafitzgerald · Pull Request #709 · NCAR/geocat-comp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Support updates #709

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
merged 3 commits into from
Apr 1, 2025
Merged

Support updates #709

merged 3 commits into from
Apr 1, 2025

Conversation

kafitzgerald
Copy link
Contributor
@kafitzgerald kafitzgerald commented Apr 1, 2025

PR Summary

The motivation for this PR was to remove the reference to the old ncar/geocat repo as a suggested support mechanism. I opted to do that by adjusting the language and recommendations on our longer format support page. This allows us to keep that additional content, but isn't consistent with the simplified approach used in geocat-applications or geocat-examples that just links out to GitHub issues and the GeoCAT suggestion box. I'm inclined to keep at least viz and comp consistent, but don't have too strong of an opinion otherwise. Curious what others think.

Currently, this PR does the following:

  • Removes reference to ncar/geocat repo from support page
  • Minor updates to support page language
  • Changes "feature request form" to "suggestion box" for consistency

Related Tickets & Documents

Closes #706

PR Checklist

General

  • PR includes a summary of changes
  • Link relevant issues, make one if none exist
  • Add a brief summary of changes to docs/release-notes.rst in a relevant section for the upcoming release.
  • Add appropriate labels to this PR
  • PR follows the Contributor's Guide

@kafitzgerald kafitzgerald added the documentation Improvements or additions to documentation label Apr 1, 2025
@anissa111
Copy link
Member

I'm inclined to keep at least viz and comp consistent, but don't have too strong of an opinion otherwise. Curious what others think.

I also think we should probably keep consistency over viz and comp, but don't have strong opinions.

@kafitzgerald
Copy link
Contributor Author

Alright, in that case I'll go ahead and merge and make the corresponding changes on viz.

If we feel like dropping the support page and moving to links in the future - not opposed.

@kafitzgerald kafitzgerald merged commit 5cb2454 into NCAR:main Apr 1, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove reference to ncar/geocat repo on support page
3 participants
31A1
0