CSC510-2015-Axitron / maze

Main repo for maze game
0 stars 2 forks source link

No escape from designer #79

Closed meneal closed 9 years ago

meneal commented 9 years ago

Once you're in you're trapped in there! Could use some sort of button to get back out to playing the game.

duh102 commented 9 years ago

Are you taking care of this?

meneal commented 9 years ago

Not currently. I'm still trying to figure out how to get the user id once the user is logged in. I just noticed and figured there should be some sort of way back to playing the game.

duh102 commented 9 years ago

Ok, I'll take care of it. You can check if the user's logged in by checking remoteDB.isLogon, and then if so, the user's id is remoteDB.userID .

meneal commented 9 years ago

Thing about that is when to actually check for the mazes. So first thing when the game loads up you won't be logged in. I'm really just trying to figure out at what point to check whether they've logged in and then pull the id number. I'm trying adding it in at various points after the login takes place.

On Sat, Mar 28, 2015 at 8:37 PM, Will Morrow notifications@github.com wrote:

Ok, I'll take care of it. You can check if the user's logged in by checking remoteDB.isLogon, and then if so, the user's id is remoteDB.userID .

— Reply to this email directly or view it on GitHub https://github.com/CSC510-2015-Axitron/maze/issues/79#issuecomment-87335139 .

duh102 commented 9 years ago

Why not at every time they click the levels button? You'd need to update the levels completed every time they open it anyway, in case they've completed a level in the meantime.

meneal commented 9 years ago

I actually just put it in the callback on line 720. It's just a function that builds the entire current user list whenever they log in. Do you think that's a bad place for it? If i update it when the levels button is pressed then we have two api calls instead of one every time the levels button is pushed...

On Sat, Mar 28, 2015 at 8:41 PM, Will Morrow notifications@github.com wrote:

Why not at every time they click the levels button? You'd need to update the levels completed every time they open it anyway, in case they've completed a level in the meantime.

— Reply to this email directly or view it on GitHub https://github.com/CSC510-2015-Axitron/maze/issues/79#issuecomment-87335292 .

duh102 commented 9 years ago

Just thinking that since it changes every time they complete a level, you might want to update it when they open the levels bar, but we can push that off to theoretical polish.