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

Updated dependencies to support API 6702 #236

Merged
merged 3 commits into from
Jul 22, 2017
Merged

Conversation

deivuh
Copy link
Contributor

@deivuh deivuh commented Jul 16, 2017

No description provided.

@deivuh deivuh mentioned this pull request Jul 16, 2017
@ghost
Copy link

ghost commented Jul 16, 2017

How do i get this to work now?

@deivuh deivuh force-pushed the master branch 3 times, most recently from bc658df to 2388b21 Compare July 18, 2017 06:53
@YesThatAllen
Copy link
Contributor

I was able to use

yarn
yarn run dev

on this branch, and it allowed me to log in (I did nothing else past that)

@ghost
Copy link

ghost commented Jul 19, 2017

It doesn't even allow me to start the app.

@YesThatAllen
Copy link
Contributor

@therealabradolf if you're trying to compile the app from this branch, let us know specifically what you've tried and what errors you are seeing.

This built on my Mac running 10.12, I'm sure once we have confirmation that it's working on other platforms, @vinnymac will be able to merge & post a release for general usage.

PS my steps were to check out deivuh's branch, and in terminal I brew-upgraded then

yarn
yarn run dev
changed the api to 6702

Of course, there's a new API version releasing today, so there's no telling if this will continue to work.. it is out of our control.

@vinnymac
Copy link
Owner

I am gonna merge this, but we need 6900 now anyways.

@vinnymac vinnymac merged commit 0886ba1 into vinnymac:master Jul 22, 2017
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