search

OpenSprites / OpenSprites-Origin

The original OpenSprites website, written in PHP. Discontinued - remains for historical reasons. See OpenSprites-Next for the new generation of Scratch resource sharing.
MIT License
19 stars 9 forks source link

Selfhosting directs to opensprites.org ._. #207

Closed QuillzLouis closed 8 years ago

109C commented 8 years ago

How is this not a bug? It doesn't make any sense for it to tell the client to go to some other website.

jamesd-uk commented 8 years ago

He's talking about hosting a development copy locally, by the way.

No OS developer here uses a local copy because the whole website is specifically built around our setup and getting every link on the website to point to its root location instead of the OS domain isn't worthwhile.

QuillzLouis commented 8 years ago

It still is worthwhile what if you change domain? Find and replace could fix this. On 21 Mar 2016 9:58 p.m., "James" notifications@github.com wrote:

He's talking about hosting a development copy locally, by the way.

No OS developer here uses a local copy because the whole website is specifically built around our setup and getting every link on the website to point to its root location instead of the OS domain isn't worthwhile.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/OpenSprites/OpenSprites/issues/207#issuecomment-199504814

jamesd-uk commented 8 years ago

Why would we ever change domain names?

QuillzLouis commented 8 years ago

You never know.

matthewr6 commented 8 years ago

Honestly, I agree with Quillz, if I'm reading this correctly - are you saying to not develop locally?

QuillzLouis commented 8 years ago

That would sorta work, l am suggesting a way to instantly test your code. On 21 Mar 2016 10:14 p.m., "Matthew" notifications@github.com wrote:

Honestly, I agree with Quillz, if I'm reading this correctly - are you saying to not develop locally?

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/OpenSprites/OpenSprites/issues/207#issuecomment-199512509

matthewr6 commented 8 years ago

unittestsunittestsunittestsunittests

QuillzLouis commented 8 years ago

Wut the hell On 21 Mar 2016 10:18 p.m., "Matthew" notifications@github.com wrote:

unittestsunittestsunittestsunittests

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/OpenSprites/OpenSprites/issues/207#issuecomment-199513848

jamesd-uk commented 8 years ago

https://lmddgtfy.net/?q=unittest

It's important to note that to even test anything that requires an account locally, both databases and the version of EsoTalk we use would need to be duplicated.

matthewr6 commented 8 years ago

Oh yeah you guys are using straight PHP aren't you lol This is a great example of technical debt :P

QuillzLouis commented 8 years ago

A python library? I didn't think open sprites used python? On 21 Mar 2016 10:33 p.m., "Firedrake969" notifications@github.com wrote:

Oh yeah you guys are using straight PHP aren't you lol This is a great example of technical debt :P

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/OpenSprites/OpenSprites/issues/207#issuecomment-199518869

matthewr6 commented 8 years ago

Where did Python come into this? o.o

QuillzLouis commented 8 years ago

Unittest

matthewr6 commented 8 years ago

That's not exclusive to Python :P

bates64 commented 8 years ago

To be honest the best way to fix these bugs is a rewrite. /controversial

jamesd-uk commented 8 years ago

^ pls rewrite in node kthxbye

matthewr6 commented 8 years ago

that wouldn't be that bad :P

QuillzLouis commented 8 years ago

Pls rewrite in English merci