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

Update ember-basic-dropdown to remove deprecation warning #1179

Closed

Conversation

ohcibi
Copy link

@ohcibi ohcibi commented Feb 9, 2021

The warning was caused by an outdated version of ember-maybe-in-element.

The warning was caused by an outdated version of ember-maybe-in-element.
@bjornharrtell
Copy link
Contributor

Another reason to merge this is that ember-basic-dropdown 2.x is broken on ember-source 3.25.0 and forward. See cibernox/ember-basic-dropdown#578 (comment).

@bertdeblock
Copy link

ember-basic-dropdown should remain under dependencies though? It's used directly here.

@bjornharrtell
Copy link
Contributor

@bertdeblock I would think it should work and is more appropriate as a dev dependency? (Used in build not directly in runtime)

@bertdeblock
Copy link

Don't think so? If it would work listed under devDependencies that would probably be because ember-paper also depends on ember-power-select, which in turn depends on ember-basic-dropdown (and there it's listed under normal dependencies as well). ember-paper uses the <BasicDropdown> component directly so keeping it under dependencies is the right thing.

@bertdeblock
Copy link

I think you are confusing it with listing dependencies under devDependencies for a project?

@bjornharrtell
Copy link
Contributor

@bertdeblock ah yes you are of course correct, my mistake. And I think it does not make much sense to update only one of these related components. I've started on #1182 but it's not a trivial upgrade.

@bjornharrtell
Copy link
Contributor

Discussed with @miguelcobain and since ember-paper is essentially in maintenance only mode it is better to solve this issue with a patch for ember-basic-dropdown v2 branch instead of attemping a large refactoring in ember-paper.

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.

3 participants