Skip to content

Conversation

@abdulrahmanelnory1
Copy link
Contributor

@abdulrahmanelnory1 abdulrahmanelnory1 commented Dec 17, 2025

Remove adoption features from SAS homepage and unpublish adoption pages

Changes Made

  • Commented out all external references to adoptananimal.jsp across the codebase
  • Commented out all external references to createadoption.jsp across the codebase
  • Removed "Adopt a Shark" buttons from index.jsp
  • Deleted "How Can I Help?" section from index.jsp

Files Modified

  • adoptionform.jsp - Commented CSS link to createadoption.css and redirect to createadoption.jsp
  • adoptions/emailAlterAdoption.jsp - Commented CSS link to createadoption.css
  • confirmSubmit.jsp - Commented link to adoptananimal.jsp
  • header.jsp - Commented navigation links to adoptananimal.jsp and CSS link to createadoption.css
  • index.jsp - Deleted adoption button and "How Can I Help?" section
  • individualAdoptionEmbed.jsp - Commented link to createadoption.jsp
  • oneTimeDonation.jsp - Commented CSS link to createadoption.css

User Impact

  • Adoption pages are no longer accessible through normal site navigation
  • All "Adopt a Shark" buttons and links have been removed from user-facing pages
  • "How Can I Help?" section with adoption information removed from homepage
  • Users cannot navigate to adoption functionality.

PR fixes #1317

Before you Submit!

  • ✅ Is all the text internationalized? - N/A (only removing/commenting existing code)
  • ✅ If you made a change to the header, did you update the react, jsp, and html? - Updated JSP header only
  • ✅ Are all dependencies at a locked version? - No new dependencies added
  • ✅ Did you adhere to best practices? - Yes
  • ✅ Is there a quick unit test you can add? - N/A (feature removal, test would verify pages are inaccessible via UI)

-Unpublish /adoptananimal.jsp and /createadoption.js
@abdulrahmanelnory1 abdulrahmanelnory1 changed the title -Remove adoption reference from SAS homepage. Remove adoption reference from SAS homepage. Dec 17, 2025
@abdulrahmanelnory1 abdulrahmanelnory1 changed the title Remove adoption reference from SAS homepage. Remove adoption reference from SAS homepage Dec 17, 2025
@JasonWildMe JasonWildMe merged commit 05235aa into WildMeOrg:ncaquariums Dec 23, 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.

2 participants