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

Fixed database connection #39

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Fixed database connection #39

wants to merge 19 commits into from

Conversation

Yohn
Copy link

@Yohn Yohn commented Jun 28, 2024

After installing we got an error on the site because of the database connection not being set in dbconnection.php

Yohn added 5 commits June 27, 2024 23:40
removed array() and made the server.php file return the config variables instead of adding more variables to the script
@Yohn
Copy link
Author

Yohn commented Jun 28, 2024

Fixed the admin login, and corrected the define.php file when it gets built to put the / after the domain names.

I like what you have done, its a nice start, but the database needs a little work with optimization, and theres a bunch of the same data in multiple different tables, and the collation for some tables and columns seems off for me. I could be wrong there, but that could be something you'll need to address later.

Now that I'm in the admin pages, where do I find the GrapeJS page builder thing at? hahaha I might be blind, I'll look again soonish.

Edit
Ok, I did find the builder, and I am very impressed. You put a lot of work into this already man, nicely done!

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