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

Use js env vars #153

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Use js env vars #153

wants to merge 14 commits into from

Conversation

MattNguyen
Copy link
Member

@MattNguyen
Copy link
Member Author

this is live.

@bsiddiqui
Copy link
Member

on line 84 or so in services.js.coffee there is a window.location.replace ... change the url to the domain var

@bsiddiqui
Copy link
Member

on line 255 or so in services.js.coffee there is a link for the FB send dialog ... change it to the domain var

@bsiddiqui
Copy link
Member

FB is not defined if this is live and it's playing the same song as a result and giving a 500 if you try and create a playlist from FB event

@MattNguyen
Copy link
Member Author

500 problem is gone.

@MattNguyen
Copy link
Member Author

fixed the other two issues and redeployed.

@MattNguyen
Copy link
Member Author

I think we talked about this before, but why would is it playing the same song if FB isn't live?

pusherKey: "#{Rails.configuration.pusher_key}"
}
};

(function(d) {
Copy link
Member Author

Choose a reason for hiding this comment

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

isn't this where facebook is defined? Would putting it above the Playedbyme definition make a difference?

Copy link
Member

Choose a reason for hiding this comment

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

it shouldn't make a difference bc neither are dependent on each other

@bsiddiqui
Copy link
Member

It's because of continuous play - since there are no preferences due to lack of FB data, we end up searching soundcloud with an empty string and playing the first song (we'll change this later)

@@ -81,7 +81,7 @@ ocarinaServices.factory 'Authentication', ['$http', ($http) ->
access_token: auth.accessToken
$.get "#{apiURL}/api/users/authenticate.json", authParams, (data) ->
Authentication.setCookie("user_id", data.id, 1)
window.location.replace "http://localhost:4400?user_id=#{data.id}"
window.location.replace "http://#{Playedbyme.env.domain}:4400?user_id=#{data.id}"
Copy link
Member

Choose a reason for hiding this comment

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

drop the :4400 - for some reason this is still redirecting me in login to localhost

@bsiddiqui
Copy link
Member

FB is still not working - I'll try and investigate tomorrow morning

@MattNguyen
Copy link
Member Author

could it be that we're using old fb permissions on staging?

@bsiddiqui
Copy link
Member

what do you mean old fb permissions?

Basil Siddiqui


@BasilKSiddiqui
bsiddiqui.com
github.com/bsiddiqui
linkedin.com/in/bsiddiqui

On Thu, Oct 10, 2013 at 12:30 PM, Matthew Nguyen
[email protected]:

could it be that we're using old fb permissions on staging?


Reply to this email directly or view it on GitHubhttps://github.com//pull/153#issuecomment-26068986
.

@MattNguyen
Copy link
Member Author

when we first signed up on the staging server, i think we were using different fb permissions than the ones we are using now.

@bsiddiqui
Copy link
Member

FB permissions are determined by the app, not what you put when registering
the app (that's just if people are looking through the FB app catalog)

On Thu, Oct 10, 2013 at 3:41 PM, Matthew Nguyen [email protected]:

when we first signed up on the staging server, i think we were using
different fb permissions than the ones we are using now.


Reply to this email directly or view it on GitHubhttps://github.com//pull/153#issuecomment-26084503
.

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.

2 participants