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

Allow groups to show in tag search results for public visitors #1615

Merged
merged 1 commit into from
Apr 20, 2023

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, so removed the filter for public users.

Brief summary of your testing
Manual testing

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

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.

LGTM.

Next time for the testing feedback, I'm looking for more of a description of what you did to test to confirm your change works as expected:

For a bug fix, something like:
"I ran the code before my fix with some input and it returned/behaved like X, after I applied my fix I re-ran the test with the same input and the expected result of (whatever expected restult) showed up properly"

OR

for a new feature, something like:
"This was a new feature, I ran tests with various inputs, range tested inputs 1,2,3,4, and made sure there were max length tests on fields 1,2,3,4. Logged out, feature was unavailable (this is the expected behavior). Talked with client, they took a look at it and said it was fine, or I went through the design doc (section xxx) to make sure all design criteria were properly"

@dbenham dbenham merged commit 7f33359 into dev Apr 20, 2023
@nkissebe nkissebe deleted the taggedGroupsShowInPublicSearch branch April 18, 2024 23:22
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