snippet-library icon indicating copy to clipboard operation
snippet-library copied to clipboard

Fixed issue where fields using the JSON API object without filters returned no results.

Open spivurno opened this issue 2 years ago • 2 comments
trafficstars

Context

⛑️ Ticket(s): https://secure.helpscout.net/conversation/2377701857/55479/

Summary

This PR addresses an issue introduced in https://github.com/gravitywiz/snippet-library/pull/677 where fields populating from the JSON API object without filters return no results. The issue was that we skipped that filter rather than indicating that all results matched that filter.

Worth noting this PR does not address the fact that the default limit of 501 is not honored by this snippet.

spivurno avatar Oct 04 '23 20:10 spivurno

Fails
:no_entry_sign:

Pull request title does match the correct format. The Pull Request title should match our Snippet Library Pull Request Title Guidelines in Notion.

:no_entry_sign:

Commit message 'Fixed issue where fields using the JSON API object without filters returned no results.' does match the correct format. See our Snippet Library Commit Messages Guidelines in Notion.

Generated by :no_entry_sign: dangerJS against 5247f936ddb9a78bc1cb830cb0a832b48b65afcb

github-actions[bot] avatar Oct 04 '23 20:10 github-actions[bot]

Fails
:no_entry_sign:

Pull request title does match the correct format. The Pull Request title should match our Snippet Library Pull Request Title Guidelines in Notion.

:no_entry_sign:

Commit message 'Fixed issue where fields using the JSON API object without filters returned no results.' does match the correct format. See our Snippet Library Commit Messages Guidelines in Notion.

Generated by :no_entry_sign: dangerJS against 5247f936ddb9a78bc1cb830cb0a832b48b65afcb

github-actions[bot] avatar Oct 04 '23 20:10 github-actions[bot]

Fixed by some other PR. 😅

spivurno avatar Jan 10 '25 11:01 spivurno