landgreen / n-gon

2-d physics rogue-lite platformer shooter
https://landgreen.github.io/n-gon/
GNU General Public License v3.0
134 stars 193 forks source link

github pages trouble #212

Closed AnnonymousNerd87 closed 2 months ago

AnnonymousNerd87 commented 2 months ago

I recently forked n-gon and made a few dumb changes (the physics for some stuff is different), while removing some smart ones (ammo management, death), and everything was running smoothly. I had made a github pages page (?) and it had worked before, but after committing the changes and waiting for the upload the github pages link went down for a while, then when it came back up only the index.html loaded, so when I tried to start the game nothing happened. I didn't even touch the index.html until then so I don't know what went wrong. Here is the link to the repo: https://github.com/AnnonymousNerd87/AnnonymousNerd87.github.io I have already invited you as a collaborator so we can try to fix it.

not expecting a response soon, I know school days aren't the best time.

landgreen commented 2 months ago

The first thing I notice is that you don't have a player.js file

kgurchiek commented 2 months ago

I think the issue is the extra closing bracket on line 424 of player.js

AnnonymousNerd87 commented 2 months ago

I have a player.js file, I just updated it a bit.

AnnonymousNerd87 commented 2 months ago

I can restore it now, see if it helps

kgurchiek commented 2 months ago

There's a syntax error in your modified player.js preventing it from running

landgreen commented 2 months ago

Where is your player.js file? (I'm in github pages)

Screenshot 2024-04-20 at 10 52 04 AM
kgurchiek commented 2 months ago

@landgreen it isn't loading due to a syntax error, but it does exist https://github.com/AnnonymousNerd87/AnnonymousNerd87.github.io/blob/master/js/player.js

AnnonymousNerd87 commented 2 months ago

I just restored the wipe canvas function I accidentally deleted, testing now

AnnonymousNerd87 commented 2 months ago

nope, still not working

AnnonymousNerd87 commented 2 months ago

let me revert the whole player.js file to an original version

AnnonymousNerd87 commented 2 months ago

I didn't make too much edits in my version, so it should help

landgreen commented 2 months ago

@landgreen it isn't loading due to a syntax error, but it does exist https://github.com/AnnonymousNerd87/AnnonymousNerd87.github.io/blob/master/js/player.js

Oh so if you have a syntax error it's hidden in dev tools? I didn't know that.

kgurchiek commented 2 months ago

Yeah, if scripts fail to run it won't show them in the sources tab

AnnonymousNerd87 commented 2 months ago

I just fixed the syntax error, saving now

AnnonymousNerd87 commented 2 months ago

it works!

AnnonymousNerd87 commented 2 months ago

thanks landgreen and kgurchiek, I hadn't noticed the syntax error

AnnonymousNerd87 commented 2 months ago

this saved me a lot of trouble

AnnonymousNerd87 commented 2 months ago

Screen recording 2024-04-20 11.04.46 AM.webm

AnnonymousNerd87 commented 2 months ago

man I love indie devs