Skip to content

Show tagged groups Public Users#1685

Merged
dbenham merged 2 commits intodevfrom
tagSearchGroupVisiblePublic
Nov 14, 2023
Merged

Show tagged groups Public Users#1685
dbenham merged 2 commits intodevfrom
tagSearchGroupVisiblePublic

Conversation

@jessewoo
Copy link
Contributor

Source JIRA card(s) and hubzero ticket(s)

Brief summary of the issue
When a user adds a tag to a group, and upon searching for a tag, that specific group won't appear in the search results for public visitors but will appear for registered and logged in users.

An example is on the search results of https://theghub.org/tags/ghub. The ISMIP6 group has the associated "Ghub" tag. But that ISMIP6 group only shows up in the results when the user is registered and logged in. The issue is how we can make groups show up for specific tags in the search results for non-registered (public) users.

Brief summary of the fix

  • The search SQL statement originally filtered out those groups. Talking it over, Nick added conditions to removed the "from" from guests and users in role that view all groups (admin).

Brief summary of your testing
Manual testing locally. It works where group shows up logged in or public.

Do the change needs to be hotfixed to any production hubs before a normal core rollout
No

Double check someone is assigned to review the ticket
Yes - Nick and David

@jessewoo jessewoo self-assigned this Oct 26, 2023
@jessewoo
Copy link
Contributor Author

@dbenham - could we review this in our next dev meeting?

@jessewoo
Copy link
Contributor Author

@nkissebe - thank you for the sql query

@jessewoo
Copy link
Contributor Author

jessewoo commented Nov 8, 2023

@dbenham - ready to be merged.

Copy link
Contributor

@dbenham dbenham left a comment

Choose a reason for hiding this comment

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

Group reviewed on Nov 8th, only feedback was whitespace which was addressed.

@dbenham dbenham merged commit 91df590 into dev Nov 14, 2023
dbenham pushed a commit that referenced this pull request Dec 20, 2023
@nkissebe nkissebe deleted the tagSearchGroupVisiblePublic branch April 18, 2024 23:09
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

Comments