CSC510-2015-Axitron / maze

Main repo for maze game
0 stars 2 forks source link

DB needs real data #54

Closed meneal closed 9 years ago

meneal commented 9 years ago

So right now the db has nonsense data in it from testing. We should clean it out and put in the and some base user.

duh102 commented 9 years ago

Should we retain the testing users, or remove them as well? I can go in and manually delete users with no issue. I also just cleared out the testing mazes (And all the duplicate ones).

duh102 commented 9 years ago

I can start working on porting the existing levels over, do we want to include all of them or only a smattering?

meneal commented 9 years ago

Either way on the testing user. If we don't retain them we'd still need some other starter user. As far as the levels, I think for now we can include all of them, since we don't have other content to replace them really yet.

On Fri, Mar 20, 2015 at 4:45 PM, Will Morrow notifications@github.com wrote:

I can start working on porting the existing levels over, do we want to include all of them or only a smattering?

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

duh102 commented 9 years ago

I haven't given any of the levels names, but they're now all in there: http://axemaze-db.herokuapp.com/mazes/category/1 If you give me names I can happily put them in.

duh102 commented 9 years ago

Actually, I can just get some names from Firebase, I'll go do that.

meneal commented 9 years ago

Cool. That's fine. I was going to say just use a fantasy name generator or something. I just made those fb names up really quick.

On Fri, Mar 20, 2015 at 10:25 PM, Will Morrow notifications@github.com wrote:

Actually, I can just get some names from Firebase, I'll go do that.

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

duh102 commented 9 years ago

Well, I'll call this closed, when the client hookups with the API work we can add more real maze data.

macluo commented 9 years ago

Is there a reason the first maze begins with id 19? Need to know this in case 0-18 would be reserved for something else.

duh102 commented 9 years ago

Nope, they auto-index starting at 1 and we had 18 junk mazes (the first two and a bunch of submissions from testing).

On Sun, Mar 22, 2015 at 3:08 PM, Mark Luo notifications@github.com wrote:

Is there a reason the first maze begins with id 19? Need to know this in case 0-18 would be reserved for something else.

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

macluo commented 9 years ago

Okay. Thanks.

meneal commented 9 years ago

http://axemaze-db.herokuapp.com/maze/1, returns this:

`{ "response": "maze not found", "query": "1"

}`

And when you run http://axemaze-db.herokuapp.com/mazes/category/1, you get back the following:

{ "category": 1, "categoryName": "Small Mazes (5-10)", "mazes": [ { "mazeno": 19, "displayName": "Dungeon of Smallness" }, { "mazeno": 20, "displayName": "Dungeon of Less Smallness" }, { "mazeno": 21, "displayName": "Dungeon of the Ogre" }, { "mazeno": 22, "displayName": "Dungeon of the Goblin" }, { "mazeno": 23, "displayName": "Dungeon of the Dwarf" }, { "mazeno": 24, "displayName": "Dungeon of the Warlock" }, { "mazeno": 25, "displayName": "Dungeon of the Imp" }, { "mazeno": 26, "displayName": "Dungeon of the Troll" } ] }

If the mazes should be ordered starting with one then I would think the first maze in category 1 should be mazeno one right?

On Sun, Mar 22, 2015 at 3:12 PM, Mark Luo notifications@github.com wrote:

Okay. Thanks.

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

duh102 commented 9 years ago

They're auto-indexed, meaning they started numbering at 1, but I deleted 18, so the only remaining ones are id 19+.

On Sun, Mar 22, 2015 at 3:15 PM, Matthew Neal notifications@github.com wrote:

http://axemaze-db.herokuapp.com/maze/1, returns this:

`{ "response": "maze not found", "query": "1"

}`

And when you run http://axemaze-db.herokuapp.com/mazes/category/1, you get back the following:

{ "category": 1, "categoryName": "Small Mazes (5-10)", "mazes": [ { "mazeno": 19, "displayName": "Dungeon of Smallness" }, { "mazeno": 20, "displayName": "Dungeon of Less Smallness" }, { "mazeno": 21, "displayName": "Dungeon of the Ogre" }, { "mazeno": 22, "displayName": "Dungeon of the Goblin" }, { "mazeno": 23, "displayName": "Dungeon of the Dwarf" }, { "mazeno": 24, "displayName": "Dungeon of the Warlock" }, { "mazeno": 25, "displayName": "Dungeon of the Imp" }, { "mazeno": 26, "displayName": "Dungeon of the Troll" } ] }

If the mazes should be ordered starting with one then I would think the first maze in category 1 should be mazeno one right?

On Sun, Mar 22, 2015 at 3:12 PM, Mark Luo notifications@github.com wrote:

Okay. Thanks.

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

.

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

meneal commented 9 years ago

Gotcha. That makes sense.

On Sun, Mar 22, 2015 at 3:19 PM, Will Morrow notifications@github.com wrote:

They're auto-indexed, meaning they started numbering at 1, but I deleted 18, so the only remaining ones are id 19+.

On Sun, Mar 22, 2015 at 3:15 PM, Matthew Neal notifications@github.com wrote:

http://axemaze-db.herokuapp.com/maze/1, returns this:

`{ "response": "maze not found", "query": "1"

}`

And when you run http://axemaze-db.herokuapp.com/mazes/category/1, you get back the following:

{ "category": 1, "categoryName": "Small Mazes (5-10)", "mazes": [ { "mazeno": 19, "displayName": "Dungeon of Smallness" }, { "mazeno": 20, "displayName": "Dungeon of Less Smallness" }, { "mazeno": 21, "displayName": "Dungeon of the Ogre" }, { "mazeno": 22, "displayName": "Dungeon of the Goblin" }, { "mazeno": 23, "displayName": "Dungeon of the Dwarf" }, { "mazeno": 24, "displayName": "Dungeon of the Warlock" }, { "mazeno": 25, "displayName": "Dungeon of the Imp" }, { "mazeno": 26, "displayName": "Dungeon of the Troll" } ] }

If the mazes should be ordered starting with one then I would think the first maze in category 1 should be mazeno one right?

On Sun, Mar 22, 2015 at 3:12 PM, Mark Luo notifications@github.com wrote:

Okay. Thanks.

— Reply to this email directly or view it on GitHub <

https://github.com/CSC510-2015-Axitron/maze/issues/54#issuecomment-84681057

.

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

.

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

macluo commented 9 years ago

The first id being 19 is not an issue. The script will first get whatever parameters they need from the remote db before it sets up the maze.

macluo commented 9 years ago

I see some data are in arrays already. This actually saves me a lot of work. lol

meneal commented 9 years ago

Good, so the apiClient is working for you?

On Sun, Mar 22, 2015 at 3:21 PM, Mark Luo notifications@github.com wrote:

I see some data are in arrays already. This actually saves me a lot of work. lol

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

macluo commented 9 years ago

I am lazy so …. I am trying to avoid reading code. LOL For loading the mazes, I read the output from Heroku site and then improvise. I would use apiClient for authentication and others.

macluo commented 9 years ago

There is a very small typo in the mazeJSON string of maze 19.

\"board\":[[6,5,6],[12,1,8]]}"

It should be

\"board\":[[6,5,3],[12,1,8]]}"

This minor error actually blocks the exit. The knight now can't go anywhere.

duh102 commented 9 years ago

Should be fixed, I updated it in the DB.