Fix: allow unlimited results for question:* taglocations search#11848
Open
krupali-narania wants to merge 1 commit intopubliclab:mainfrom
Open
Fix: allow unlimited results for question:* taglocations search#11848krupali-narania wants to merge 1 commit intopubliclab:mainfrom
krupali-narania wants to merge 1 commit intopubliclab:mainfrom
Conversation
|
Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help. |
Author
|
Hi! Thank you for the warm welcome 😊
This PR is based on the discussion around improving `taglocations` behavior
for `question:*`.
There wasn’t an existing issue created for this, but I’m happy to link one
or update the PR if needed.
Please let me know if you’d like me to open an issue and reference it here.
Thanks!
…On Sat, 17 Jan 2026 at 23:49, welcome[bot] ***@***.***> wrote:
*welcome[bot]* left a comment (publiclab/plots2#11848)
<#11848 (comment)>
Thanks for opening this pull request! This space is protected by our Code
of Conduct <https://publiclab.org/conduct> - and we're here to help.
Dangerbot will test out your code and reply in a bit with some pointers
and requests.
Also please refer here <#3840>
for installation help 💿
There may be some errors, *but don't worry!* We'll work through them with
you! 👍🎉😄
One thing that can help to get started is to make sure you've included a
link back to the original issue you're solving, in the format fixes #0000
(for example). And to make sure the PR title describes what you're trying
to do! (often it can be the same as the issue title) Thanks! 🙌
Then, you can say hello in our chatroom
<https://gitter.im/publiclab/publiclab> & share a link to this PR to get
a review! 👋 ✅
—
Reply to this email directly, view it on GitHub
<#11848 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BJVXIT4L7MIGFEJBRHDVWO34HJ4JXAVCNFSM6AAAAACSAVDH46VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTONRUGE3TSNZWGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates the
taglocationssearch behavior to return all results when the tag query isquestion:*, ensuring complete result sets instead of being limited.Related to taglocations search behavior for
question:*Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test@publiclab/reviewersfor help@publiclab/reviewers
Hi! This is my first contribution to Public Lab.
This PR updates the taglocations search to return all results for
question:*.Please let me know if any changes or improvements are needed — happy to iterate.
Thank you!