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

v3 Proposal #341

Closed
mixonic opened this issue Dec 6, 2021 · 8 comments
Closed

v3 Proposal #341

mixonic opened this issue Dec 6, 2021 · 8 comments

Comments

@mixonic
Copy link
Contributor

mixonic commented Dec 6, 2021

In alignment with Addepar/ember-table#919 (ember-table is one of the main consumers of vertical collection), I propose a v3 major release of vertical collection.

The main breaking change is: Remove support for Ember < 2.18 and ensure compatibility with Ember 4.

Thoughts?

@mixonic
Copy link
Contributor Author

mixonic commented Dec 7, 2021

The PR at #343 tracks a prerelease branch. CI is set up, and I'll look to extract whatever minimal changes from #335 that I can get it green.

I've also pinned the minimum Node version at 14.

@mixonic
Copy link
Contributor Author

mixonic commented Dec 9, 2021

3.0.0-0 has been released, and a release-2 branch for legacy v2 support has been created.

Downstream in Ember Table I've created Addepar/ember-table#921 as a smoke test, though it doesn't run tests against Ember 4 at this point.

Please try the pre-release if applicable, and provide feedback!

@runspired
Copy link
Contributor

@mixonic I think we forgot to actually publish the v3

@mixonic
Copy link
Contributor Author

mixonic commented Feb 24, 2022

@runspired my plan was to wait until ember-table gets a stable release well tested via integration into real apps before cutting a stable here.

@bakerac4
Copy link

bakerac4 commented Apr 7, 2022

Can this release be cut now at this point? Or are we still waiting for results from ember-table?

@mixonic
Copy link
Contributor Author

mixonic commented Apr 7, 2022

I've been trying to get the last beta of ember-table successfully landed into a large app at Addepar. We're seeing unexpected width changes in cells. At first it seemed to be from the 3.0.0-0 to 3.0.0-1 changes, however I believe that analysis might have been in error and I'm debugging.

I suspect we're in the clear on the vertical collection from, though my confidence is only 80%. I'll be able to allocate some time over the next two days to try and get the style change isolated or rationalized so we can ship.

@runspired
Copy link
Contributor

I did end up trying to use this in an app as well and found a few problems, though not necessarily caused by vertical-collection itself, and also not necessarily due to the v3 bump since we were pinned to a commit that was from a while back. For instance, if we (yes very bad) mutate objects in an array while filtering over them, and that filtered array is provided to vertical-collection, it will now blow up when it didn't before. Personally I don't think that's VC's problem since its a dastardly bad thing to do, but if lots of folks start hitting issues like that we may want to at least understand what changed.

@mixonic
Copy link
Contributor Author

mixonic commented Jun 3, 2022

3.0.0 was released in early May. The issues we had been encountering with unexpected layout changed were related to a CSS bug fixed in 2.0.1 which hadn't been adopted into that codebase.

Closing this. Onward to 4.0!

@mixonic mixonic closed this as completed Jun 3, 2022
@mixonic mixonic mentioned this issue Jun 3, 2022
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

No branches or pull requests

3 participants