Skip to content

Remove ability to like pages #147

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

Closed
jfrank-nih opened this issue Mar 18, 2025 · 2 comments · Fixed by #196
Closed

Remove ability to like pages #147

jfrank-nih opened this issue Mar 18, 2025 · 2 comments · Fixed by #196

Comments

@jfrank-nih
Copy link
Member

jfrank-nih commented Mar 18, 2025

We want to remove the ability to like pages, and the text that states "be the first to like this page". This should just be temporarily commented out, rather than removed entirely.

Additionally, this row on the page also shows a counter of the number of views and comments for a given page on the news articles template. Please also remove these counters. (These were previously removed on blogs and do not appear in internal pages.)

We will need to track it down and hide these "like" options in the widget views for:

  • Blogs
  • Internal Pages
  • News Articles

Previous Step Completed:
James contacted Akumina (ticket is 14972) and learned about a configuration that would allow users to toggle this on/off, but it was set to on by default and that couldn't be changed. Due to that, we are proceeding with this ticket.

@welshja
Copy link

welshja commented Apr 28, 2025

Per discussion with Kenneth, we should also remove the view and comment counters shown on News pages. These were previously removed on blog posts. Removing on both will keep more consistency across content.

jfrank-nih added a commit that referenced this issue Apr 28, 2025
@jfrank-nih jfrank-nih reopened this Apr 29, 2025
@alinai
Copy link

alinai commented Apr 29, 2025

Looks good on pre-prod. Thanks @jfrank-nih

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants