Clarify sorting for relational fields in the Content Manager list view

[gurdiga]

What does it do?

Removes a few words that seem to cause confusion.

Why is it needed?

Because higher in that note it is said that “Only one field can be displayed per relational field.”

Related issue(s)/PR(s)

None.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
documentation	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 4, 2024 5:28pm

[strapi-cla]

All committers have signed the CLA.

[pwizla]

Hi @gurdiga!

Thank you for submitting this PR. Could you please elaborate on why you think this change is necessary and what you are trying to achieve?

I double-checked with engineers and what is currently described is expected, so at least this line shouldn't be changed in the documentation.

Here's a screenshot of what works:

Or maybe is there another part of the documentation that you would like to be improved? 🤔

[gurdiga]

Here is the place in documentation that I’m referring to:

This Note refers to “Relational fields”, first some specificities, then some limitations. The first limitation says:

Sorting cannot be enabled for relational fields which display several fields.

…which seems to imply that there can be relational fields which display several fields, while the first specificity says they can’t.

Maybe I’m misreading it. 🤔

[pwizla]

Hmm I see. This is not entirely clear for me either to be honest, so I'll dig deeper, test things further, and come back to you a.s.a.p.

[pwizla]

(still trying to figure out how it works and reproducing it, I'll keep you posted next week)

[pwizla]

Finally getting back to you.

I can't figure out why the "Sorting cannot be enabled for relational fields which display several fields." was written in the first place. It doesn't seem like it works this way, in Strapi 5 at least.

However, updating docs as you suggest would also be wrong because you can sort by relational fields, as shown in the screenshot.

So I think the docs should be updated as I suggest in my comments, actually deleting the first limitation entirely and rewriting the sentences above to simplify this.

Would that be OK for you?

[gurdiga]

Would that be OK for you?

Sure! I hope I got it right this time.

PS: Thank you for the hand-holding. 🙂

[pwizla]

Much welcome, @gurdiga!

The PR is now approved, merged, should be live on docs.strapi.io in a few minutes, and will be mentioned in the next weekly snapshot release.