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

Create if-feature "sort" #28

Open
avtobiff opened this issue Nov 14, 2024 · 1 comment · May be fixed by #29
Open

Create if-feature "sort" #28

avtobiff opened this issue Nov 14, 2024 · 1 comment · May be fixed by #29

Comments

@avtobiff
Copy link
Contributor

During review it was raised that an if-feature "sort"
should be created for parameters "sort-by", "locale",
and "direction".

All servers might not implement this and if so should
be able to indicate this to clients.

@kwatsen
Copy link
Contributor

kwatsen commented Nov 14, 2024

I have two minds about this.

On one hand, I feel that if there is a feature statement for one, then there should be a feature statement for all.

On the other hand, already is a mechanism to disable some of these for CF lists, and it’s unclear why it’s not easy to implement for CT lists (given that already servers must be able to have all config in memory, and locale is optional?

In any case, let’s discuss how best to respond on the list.

avtobiff added a commit that referenced this issue Nov 22, 2024
This feature indicates support for "locale", "sort-by", and "direction".

Fixes: #28
@avtobiff avtobiff linked a pull request Nov 22, 2024 that will close this issue
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 a pull request may close this issue.

2 participants