| Forum Index > TL Mafia |
|
Half the Sky
Germany9029 Posts
On August 25 2015 21:18 rsoultin wrote: i think we broke this  againI saw the voting thread. Onegu needed to have put the Day 3 post right after the Night 2 post. As it is, anyone that is between the first and second Night 2 posts will not have their votes counted. In general, you just do those posts consecutively right after EoD unless you know for some reason you have night voting which no normal will ever have and even few themed games will. | ||
|
Xatalos
Finland9675 Posts
| ||
|
Xatalos
Finland9675 Posts
 ay 3 | ||
|
rsoultin
Netherlands15308 Posts
| ||
|
Xatalos
Finland9675 Posts
| ||
|
Keirathi
United States4679 Posts
On August 25 2015 19:16 Xatalos wrote: Okay  Assuming you've made a fork, you can then just copy your changes to the actual forked folder and make a pull request. If there's something you noticed to fix/add, I mean.Right now there are a couple of issues remaining that could be nice to fix at some point (if you wanted ideas haha): 1) Ordering for the votecount targets (by the amount of voters they have) 2) Get night start games working (currently you can work around the issue by simply not making a correctly formatted N0 post, but naturally it would be nicer if everything worked automatically) 3) Some sort of authentication system (perhaps just requiring some single password for editing the database?) 4) A button for adding new players to a game or something like that Aye, #4 was the first thing I was going to do. I have the form made, I just need to figure out how to do the clojure stuff on the back end Like, I mostly understand what's going on, but it's still a little confusing about what I actually need lol.On August 25 2015 19:38 Xatalos wrote: Oh, and about the migrations script... If you want to try running your own Heroku app, then I think it went something like just running the "(migrations)" command once you've loaded the functions in migrations.clj. That should create the tables... Yea, After I pushed to heroku, i would do "heroku run lein repl" and then "(use 'lobo.core)" then "(migrations)" But that didn't seem to work. | ||
|
Xatalos
Finland9675 Posts
I'm not really 100% sure how I'd go about the "adding players", but I'd imagine you could look at how I managed to get the "delete player" and "delete game" functions working (you need a JavaScript function, a route in models.clj, and functions in player.clj and models.clj I'd assume?). | ||
|
Xatalos
Finland9675 Posts
| ||
|
Xatalos
Finland9675 Posts
| ||
|
Keirathi
United States4679 Posts
In home.clj, I added "(POST "/add-player" [name] (player/add-player name))" In player.clj, I added "(defn add-player [name] (models/add-player name))" Then in models.clj I added "(defn add-player [name] (insert player (values name)))" In the javascript, I call $.post("/add-player", player) where player is an object like {name: "keir"}. However, I get a 404 when I try to run it. I'm not entirely sure why, so it'll take some playing around. Clojure seems kind of hard to debug though | ||
|
Xatalos
Finland9675 Posts
What is the "player" even, a string, object? The other calls were like this $.post("/delete-game", {id: id}); | ||
|
Xatalos
Finland9675 Posts
| ||
|
Xatalos
Finland9675 Posts
| ||
|
Xatalos
Finland9675 Posts
| ||
|
Keirathi
United States4679 Posts
On August 26 2015 03:22 Xatalos wrote: The problem might be with the JS part if you get a 404... (wild guess) or possibly with the route? What is the "player" even, a string, object? The other calls were like this $.post("/delete-game", {id: id}); I said in the post that "player" was an object of the form {name: "keir"} How it generates the object is: var player = {"name": document.getElementById("newPlayer").value}. The newPlayer element is the input box for the player name, of course. So my call is basically exactly like yours: $.post("/add-player", {name: "keir"}); I'm nearly 100% positive that part is right. Something is wrong with the routing or handling of it in clojure though. On August 26 2015 03:29 Xatalos wrote: Wait... actually, you also need to include which game the player belongs to (game id). Yea I realized that when I actually poked around in the database. The player schema has id, game, and name. Oh herp derp, I figured out the problem with the routing. In the home.clj file that defines the routes, I forgot missed the closing parenthesis on defroutes. So my route wasn't being added Would be nice if that threw some kind of error though, instead of compiling and leaving my scratching my head as to why it wasn't working. Anyways, i added the game id to the add-player calls, etc, and it seems to be working correctly now. When I push the submid button for the form, keir gets added to the db. I'll clean it up and throw up a pull request soon. | ||
|
Xatalos
Finland9675 Posts
| ||
|
Keirathi
United States4679 Posts
There are a couple of things I still want to do: 1) Order the vote counts correctly 2) Add a button to reset the "current post" counter or whatever. Basically force the scanner to do a whole thread rescan, so if something happens again like in Personality again where the host forgets to do the phase change post in the voting thread, they can post it, then his the rescan button and the app will start over scanning from the front again. Basically, anytime you have to delete a game and re-add it is annoying. I'd like to eliminate the problems that currently make you do that. 3) Fuzzy matching. | ||
|
Xatalos
Finland9675 Posts
| ||
|
Xatalos
Finland9675 Posts
| ||
|
Keirathi
United States4679 Posts
But yea, basically everything except the gitignore stuff, anything related to db, and the system.properties file. | ||
| ||
|
|
CranKy Ducklings
WardiTV Invitational
ByuN vs Percival
Percival vs SHIN
Percival vs herO
ByuN vs SHIN
ByuN vs herO
SHIN vs herO
AI Arena Tournament
BSL Open Qualifier
Sparkling Tuna Cup
OSC
IPSL
BSL Open Qualifier
WardiTV Weekly
IntoTheTV X SOOP
[ Show More ] |
|
|
EPT
