search

Stillat / relationships

Provides automated bi-directional @statamic entry relationships
MIT License
16 stars 1 forks source link

Revisions seem not to be compatible with relationships #9

Closed j3ll3yfi5h closed 1 year ago

j3ll3yfi5h commented 1 year ago

Hi @JohnathonKoster!

Thank you very much for your Statamic work and this awesome addon!

Just came across a bug: revisions and relationships seem not to be compatible.

I'm not sure if it is a good decision to use them both in parallel anyway, but my use case seems from my perspective not so wrong:

Any thoughts or solutions?

JohnathonKoster commented 1 year ago

Oh interesting! I never thought to test it with revisions - will definitely do that this upcoming weekend. Is the addon trying to update each of the revisions, or pulling data from all historical ones, or just not working at all with revisions?

j3ll3yfi5h commented 1 year ago

Hm...found out, that I used the wrong type for the relationship.🤦🏼‍♂️

But what would be the correct behavior with a new relationship? A new revision?

JohnathonKoster commented 1 year ago

My initial feeling would be that it should create a new revision if both sides used revisions. Will need to experiment 🙂