meow56 / Kixley

Kixley
1 stars 2 forks source link

Should we use this page? #1

Closed cbp2170 closed 6 years ago

cbp2170 commented 6 years ago

Should we use this, or the old GitHub page?

cbp2170 commented 6 years ago

I also definitely don't have the right privileges in this repo, it won't let me commit without making a pull request you have to approve.

meow56 commented 6 years ago

Ah, sorry about the privilege thing. I... actually wasn't expecting you to find this, I've just been using this repo to test that the site works. Here, I'll give you privileges now.

meow56 commented 6 years ago

Okay, done. Also apparently there exists bowtiesareverycool now?

meow56 commented 6 years ago

Also, I'm gonna issue a few warnings about the code. 1) Don't leave functions hanging: function foo() { if(2 === 2) { bar(); } alert("Don't do this!"); } waitForUserInput() uses setTimeout(), which hands control back to the server to avoid an infinite loop crash. Stuff after a function call will be run the next time you request an input. Instead, put your function in requestInput directly. 2) I've had to remove a lot of easter eggs cuz that's not how buttons work. Sorry ;-; 3) I tried to rewrite the story but the final encounter isn't very good. help plz 4) Custom errors are a thing now! I don't remember if this was in the Bitbucket, but make your custom error with const err = new Error("Error message"); throw err; Any thanks for playing messages should read "Thanks for playing!" cuz that's the one I programmed in.

Anyways, I think that's all I want to tell you? I'm probably forgetting something though.

cbp2170 commented 6 years ago

блин, still no proper access. Also, maybe you could give the access to the Kixley organization we made?

meow56 commented 6 years ago

Still says that it's awaiting your response.

And sure, I can do that.

cbp2170 commented 6 years ago

Checked my email, found a thing to respond to. Looks like I have proper access now.

meow56 commented 6 years ago

cool