egg is now known as egg|zzz|egg
egg|zzz|egg is now known as egg
<Thomas> Hmm, only some changes and OpenDock should be able to handle multiple games
<Thomas> VITAS: Dunno if you saw it on Github, but would breaking compatibility with the current URLs and adding a redirect for the mod route (i.e. "/mod/<id>/<name>" gets redirected to "/<gameshort>/mod/<id>/<name>") be ok for you?
<VITAS> no i didnt
<VITAS> im ok with different urls if the old one redirect to them
<VITAS> +s
<Thomas> cool
<VITAS> the question is: do we want to be verbose or short?
<Thomas> Because passing the gameshort in the URL makes the JS way easier than getting it from the modid
<VITAS> or simply offer both (because we have to offer redirection anyways)
<VITAS> yes i know i had the same problem
<VITAS> but people can edit urls and that could lead to problematic results
<VITAS> https://c3goc.de/ <- only took me 12h to make (including admin panel, registration and everything)
<VITAS> it uses laravel and some addons. but the short timespan is impressive
<Thomas> Nice
egg is now known as egg|tea|egg
<RockyTV> Thomas don't we use the old way?
<RockyTV> get game from mod?
<RockyTV> also o/
<Thomas> When we get the game from the mod, we have to run the mod request first, and then we can run the other ones. When we get the game from the URL, we can run all requests in parallel
<Thomas> o/
<RockyTV> uh that works
<RockyTV> I'm working on the edit page again (procrastination hits me hard)
<SpaceDock> [OpenDock] StollD opened pull request #3: Add a 404 handler (master...feature/404-page) https://git.io/vdAv2
<RockyTV> uh what the heck
<RockyTV> mod route doesn't work
<RockyTV> it's something in my side
<Thomas> what happens?
<RockyTV> when you click the mod thumbnail in the front page, it redirects to /mods/id/name and then redirects to not-found
<RockyTV> oh, gameshort is invalid
<RockyTV> wut
<RockyTV> I'll reinstall the db
<RockyTV> still not workign
<RockyTV> /api/mods/kerbal-space-program/1 returns "gameshort is invalid
<RockyTV> /api/mods/kerbal-space-program/0 returns "the mod is not published"
<Thomas> have you tried to look at the db directly?
<RockyTV> no..
<RockyTV> will do that now
<RockyTV> it works
<RockyTV> they are working
<RockyTV> er, they are published and there*
<Thomas> what says their gameID?
<RockyTV> 1
<RockyTV> oh wait
<RockyTV> gameshort doesn't
<Thomas> so somehow, the relation between mod and game doesnt work
<RockyTV> gameshort doesn't work for that route, can you confirm?
<Thomas> I dont have an SDB setup atm
<Thomas> sorry
<Thomas> But I have an idea how to fix it
<RockyTV> uh I think we have to change some code in SDB
<RockyTV> app.Database.Where("short = ?", gameshort).Or("id = ?", cast.ToUint(gameshort)).Find(game)
<RockyTV> try casting to int first then fallback to string
<RockyTV> s/int/uint
<Qboid> RockyTV meant to say: try casting to uint first then fallback to string
<RockyTV> brb
<Thomas> There is no real need for a fallback, the gameshort should never be castable to an int
<Thomas> And the cast doesn't seem to be the problem, I think the relations are broken
<SpaceDock> [SpaceDock-Backend] StollD created fix-relations (+1 new commit): https://git.io/vdAfp
<SpaceDock> SpaceDock-Backend/fix-relations 83ad2ce Dorian Stoll: Try to fix relationship issues
<VITAS> Grand plan you have there
<VITAS> " Dorian Stoll: Try to fix relationship issues"
<Thomas> RockyTV: Can you check if it is still broken with the changes I made?
<Thomas> VITAS: :P
<VITAS> :D
<RockyTV> sure
egg|tea|egg is now known as egg
<RockyTV> fixed it Thomas
<Thomas> awesome
<SpaceDock> [SpaceDock-Backend] StollD opened pull request #29: Try to fix relationship issues (master...fix-relations) https://git.io/vdAkU
<SpaceDock> [SpaceDock-Backend] StollD pushed 1 new commit to master: https://git.io/vdAkT
<SpaceDock> SpaceDock-Backend/master 43a0c94 Dorian Stoll: Merge pull request #29 from KSP-SpaceDock/fix-relations...
<Qboid> Starting build #77 for job SpaceDock-Backend (previous build: SUCCESS)
<SpaceDock> [OpenDock] StollD closed pull request #3: Add a 404 handler (master...feature/404-page) https://git.io/vdAv2
<SpaceDock> [OpenDock] StollD pushed 1 new commit to master: https://git.io/vdAkL
<SpaceDock> OpenDock/master ab6d5d8 Dorian Stoll: Merge pull request #3 from KSP-SpaceDock/feature/404-page...
<Thomas> Why the heck does spaceduck have my profile picture o.O
<Qboid> Project SpaceDock-Backend build #77: SUCCESS in 2 min 50 sec: https://jenkins.tmsp.io/job/SpaceDock-Backend/77/
<Qboid> Dorian Stoll: Try to fix relationship issues
<Thomas> well, fixed it
<RockyTV> uh
<RockyTV> mods still don't exist in opendock
<RockyTV> oh right
<VITAS> strange thing to say for a modhosting site
<RockyTV> Thomas is this normal?
<RockyTV> nvm
<RockyTV> need to fix css in edit page: https://sr.ht/qU1e.png
dries007 has quit [Quit: ZNC - http://znc.in]
dries007 has joined #spacedock
egg is now known as egg|zzz|egg