Sorting attachments

[sodapopcan]

Code of Conduct

• I agree to follow this project's Code of Conduct

AI Policy

• I agree to follow this project's AI Policy, or I agree that AI was not used while creating this issue.

Is your feature request related to a problem? Please describe.

I would like to be able to set a default sort for attachments at the db level.

Describe the solution you'd like

Ash's "regular" relationship DSL has a sort option, ash_storage's relationship macros do not and I would like them to. This is convenient when loading attachments via :load.

Describe alternatives you've considered

• Sorting in Elixir.
• Adding a domain function to retrieve attachments by host id with sort option.
• Using load: sorted_attachments().

These work but default-sort would be very convenient in some cases.

Additional context

I'm happy to do the work here!

[sodapopcan]

I may be missing a better way to do this, but sorting is especially cumbersome with has_many_attached in that the, for example, photos_urls calculation it adds returns merely a list of strings. I haven't looked at the internals but sorting photos does not also sort photos_urls so I can't just zip them together.

[sodapopcan]

Other options I can think of (and I'm forever now scared of making bullet pointed lists but here I go):

• Add an option to put the calculation on the blog itself.
• Expose a helper to calculate the URL given a blob.

[zachdaniel]

I think the best answer is just to accept a sort option on the has_many_attached relationship :)

[sodapopcan]

I think I may be thinking of this wrong, but I still can't quite reconcile how to may URLs to blobs with has_many_attached. So if I have Blob -> Attachment -> Profile, and a profile has_many_attached :photos, this means Profile has photos and photos_urls. If we want to arbitrarily sort photos this means we have to map the urls to them. This is a bit awkward since *_urls just returns strings. Not that awkward, but an extra step. And of course, even though this is probably edge case but not heard of, this doesn't work if you need to paginate attachments.

Am I totally off here?

[zachdaniel]

There is a separate issue here: #5

Addressing that would be the solution 😄

[sodapopcan]

Closed in #13