Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor for localhost development #264

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

JohnRDOrazio
Copy link
Member

I didn't even make an issue, just while testing localhost development, I noticed I was not able to access three paths: /data, /schemas, and /tests. Just so happens that those three paths coincided with physical paths, there were folders of the same name. And PHP's builtin server just would not allow the router script to control the situation, it simply refused to serve those paths.
I have therefore renamed these three folders by moving them all under a single jsondata folder, and refactoring all the code that referenced these folders. And now, localhost development seems to work just fine on all paths.

I didn't even make an issue, just while testing localhost development, I noticed I was not able to access three paths: `/data`,  `/schemas`, and `/tests`. Just so happens that those three paths coincided with physical paths, there were folders of the same name. And PHP's builtin server just would not allow the router script to control the situation, it simply refused to serve those paths.
I have therefore renamed these three folders by moving them all under a single `jsondata` folder, and refactoring all the code that referenced these folders. And now, localhost development seems to work just fine on all paths.
@JohnRDOrazio JohnRDOrazio merged commit ef27dd3 into development Nov 18, 2024
1 check passed
@JohnRDOrazio JohnRDOrazio deleted the refactor-for-localhost-development branch November 18, 2024 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant