Skip to content

Conversation

@CarolineDenis
Copy link
Contributor

@CarolineDenis CarolineDenis commented Jul 24, 2024

Fixes #752

Checklist

  • Self-review the PR after opening it to make sure the changes look good
    and self-explanatory (or properly documented)
  • Add relevant issue to release milestone

Testing instructions

  • open query builder and select a tree table
  • make a search on any rank
  • add name field in the mapper line
  • enter the name of a node that has a synonym for the value
  • check the box search synonyms
  • run the query
  • verify that the node is being return as well as its synonym

@gitguardian
Copy link

gitguardian bot commented Oct 24, 2025

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@acwhite211 acwhite211 added this to the 7.12.0 milestone Oct 31, 2025
@acwhite211 acwhite211 marked this pull request as ready for review October 31, 2025 21:07
@acwhite211 acwhite211 requested review from a team and grantfitzsimmons October 31, 2025 21:08
Copy link
Collaborator

@emenslin emenslin left a comment

Choose a reason for hiding this comment

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

This does not appear to be working unless I am misunderstanding the instructions. I query on any rank -> name and set equal to a taxon that has synonyms but when I check 'search synonyms' only the one node appears.

image image

If I check 'search synonyms' in any tree aside from taxon I get an error.

11-07_15.21.mp4

Specify 7 Crash Report - 2025-11-07T21_21_43.622Z.txt

@github-project-automation github-project-automation bot moved this from 📋Back Log to Dev Attention Needed in General Tester Board Nov 7, 2025
@acwhite211
Copy link
Member

Fixed it so that it will work on all tree tables, not just Taxon. Try it out again @emenslin, thanks.

image image image

@acwhite211 acwhite211 requested a review from emenslin November 13, 2025 18:58
@CarolineDenis CarolineDenis requested review from a team and melton-jason November 18, 2025 13:44
Copy link
Collaborator

@emenslin emenslin left a comment

Choose a reason for hiding this comment

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

  • verify that the node is being return as well as its synonym

Looks good, I'm no longer getting an error when search synonyms is checked.
image

@emenslin emenslin requested a review from a team November 18, 2025 15:44
@grantfitzsimmons
Copy link
Member

@specify/ux-testing Can you please check this behavior against Specify 6 and make sure it matches?

Copy link
Contributor

@Iwantexpresso Iwantexpresso left a comment

Choose a reason for hiding this comment

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

  • verify that the node is being return as well as its synonym
image

this looks great so far!

@Iwantexpresso Iwantexpresso requested a review from a team December 10, 2025 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Dev Attention Needed

Development

Successfully merging this pull request may close these issues.

Add ability for query to find synonymized taxa

6 participants