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

WIP Octane upgrade #123

Closed
wants to merge 5 commits into from

Conversation

vaidehijoshi
Copy link

No description provided.

Upgrading to this first should make it easier to upgrade to Octane.
@jenweber
Copy link
Contributor

We determined in chat that the issue was that we need to enable jquery

@vaidehijoshi vaidehijoshi changed the base branch from master to octane May 15, 2019 20:24
Using the ember-octane-blueprint from ember-cli.
We need it to be enabled because some of the dependencies in this app
still use jquery.
"ember-qunit": "^3.4.1",
"ember-resolver": "^5.0.1",
"ember-qunit": "^4.1.2",
"ember-resolver": "^5.1.3",
"ember-simple-leaflet-maps": "^0.0.2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think if we update this, we shouldn't need jQuery (unless there is another plugin that is requiring it)

Crosslinking from ember-learn/ember-simple-leaflet-maps#2

@vaidehijoshi
Copy link
Author

Update on this: @wycats and I are pairing on this together and taking another approach. We're building the super-rentals app from scratch using the Octane blueprint and making note of our steps along the way that we can use for the upgrade guide once we have a completely idiomatic version of the super-rentals app.

@jenweber
Copy link
Contributor

@vaidehijoshi @wycats any updates?

@vaidehijoshi
Copy link
Author

vaidehijoshi commented Jun 14, 2019

We still have a bit more work to do on the app, but not too much more. We should hopefully be able to pair on it for another hour or two next week and get the Octane-idiomatic version of super-rentals wrapped up! (cc @wycats)

@vaidehijoshi
Copy link
Author

Since we've changed our strategy on how to go about doing the work of upgrading this app, I'm going to close this PR in favor of #127. Please see that PR for my work with @wycats on making super-rentals Octane-idiomatic :)

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