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

Add optional setting to prevent force-show first order column in backend #2527

Open
wants to merge 1 commit into
base: 2.8
Choose a base branch
from

Conversation

ameotoko
Copy link
Contributor

This ports contao/contao#6593 to Isotope's data container driver.

@aschempp
Copy link
Member

Can you please rebase this to the 2.9 branch? There are no plans to add more fixes to 2.8, and I think I have updated the driver for Contao 4.13 already (not sure if that is in there though).

@ameotoko
Copy link
Contributor Author

When will 2.9 be released then? Isotope always lacks compatibility with 4.13. I already accepted that I don't have to learn Contao 5 for a long time, but support for 4.13 should be ongoing IMO.

@fritzmg
Copy link
Contributor

fritzmg commented May 15, 2024

Both Isotope 2.8 and 2.9 are compatible with Contao 4.13.

@ameotoko
Copy link
Contributor Author

By "compatibility" I meant, that Isotope should not discard core's features when extending from Contao's core driver. This is currently the case in 2.8 with this issue and #2341 for example.

@fritzmg
Copy link
Contributor

fritzmg commented May 15, 2024

Yes, but there won't be another 2.8 release, hence you should rebase this against 2.9.

@ameotoko
Copy link
Contributor Author

I'm a little confused now... So, if I found another bug (which I just did), I should submit the fix against 2.9? But it means, I should be using 2.9 in production then? But it hasn't been released, it feels weird to use a dev branch in production, hence my question... Can you help me understand the process?

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 this pull request may close these issues.

None yet

3 participants