Skip to content

fix: document the 500-character employee filter query limit - #939

Open
abhishek1234321 wants to merge 2 commits into
mainfrom
codex/saved-filter-query-length
Open

abhishek1234321 wants to merge 2 commits into
mainfrom
codex/saved-filter-query-length

Conversation

@abhishek1234321

@abhishek1234321 abhishek1234321 commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

Document the employee saved-filter query limit increase from 255 to 500 characters in both requests and responses. Filter names retain their 255-character maximum.

Use an employee-specific query schema so other filter contracts remain untouched. Regenerate the admin bundle with Redocly 2.19.0. All 8 PR checks and all 11 local role bundles passed. Independent review and whitespace checks passed; admin lint findings match unchanged main.

Matching API change: https://github.com/fylein/fyle-platform-api/pull/5735. Publish this after the API change is available.

Clickup

https://app.clickup.com/t/PLACEHOLDER

@github-actions github-actions Bot added the size/XS Extra Small PR label Nov 7, 2026
@github-actions github-actions Bot added size/S Small PR and removed size/XS Extra Small PR labels Nov 7, 2026
@abhishek1234321 abhishek1234321 changed the title fix: remove outdated saved-filter query length limit fix: document the 500-character employee filter query limit Nov 7, 2026
Comment thread reference/admin.yaml
type: string
nullable: false
maxLength: 255
maxLength: 500

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Changing from 255 to 500 for employee filter query string limit

Comment thread reference/admin.yaml
query_string:
type: string
nullable: false
maxLength: 255

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Keeping saved filter name limit to 255 as before

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Small PR

Development

Successfully merging this pull request may close these issues.

1 participant