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

Update: "Other CoPs" to remove outdated communities #8223

Merged
merged 7 commits into from
Dec 5, 2024

Conversation

bonnieAcameron
Copy link
Contributor

@bonnieAcameron bonnieAcameron commented Nov 29, 2024

Summary

As a result of the TTS quarterly content review, these are related CoPs that no longer maintain active presence. I've searched for references to these CoPs elsewhere in the site and was not able to find them.

Preview

COP to Preview

Solution

My approach was to make these invisible on the site rather than to delete them. Please let me know if you prefer that I delete the file.

How To Test

  1. Ensure that the 5 CoPs are removed on the Communities landing page
  2. Confirm that any references elsewhere on the site do not exist
  3. Confirm the hide vs. remove file approach

@bonnieAcameron bonnieAcameron self-assigned this Nov 29, 2024
Copy link

🔍 Preview in Federalist

@bonnieAcameron bonnieAcameron changed the title Update related COPs Update: "Other CoPs" to remove outdated communities Nov 29, 2024
@bonnieAcameron bonnieAcameron requested review from nick-mon1 and removed request for ToniBonittoGSA November 29, 2024 18:24
Copy link
Contributor

@nick-mon1 nick-mon1 left a comment

Choose a reason for hiding this comment

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

We should keep these files and go with the hiding approach.
See my comment and this is approved.

# Controls how this page appears across the site
# 0 -- hidden
# 1 -- visible
weight: 1
weight: 0
Copy link
Contributor

Choose a reason for hiding this comment

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

@bonnieAcameron Updated to this 0 to hide it, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right!

@ToniBonittoGSA ToniBonittoGSA merged commit 7b517fa into main Dec 5, 2024
5 checks passed
@ToniBonittoGSA ToniBonittoGSA deleted the bc-update-related-cop branch December 5, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants