Skip to content

Comments

Enhance connection string update instructions in README#1142

Merged
hishamco merged 8 commits intosimplcommerce:masterfrom
JayanthT7:patch-2
Feb 18, 2026
Merged

Enhance connection string update instructions in README#1142
hishamco merged 8 commits intosimplcommerce:masterfrom
JayanthT7:patch-2

Conversation

@JayanthT7
Copy link
Contributor

@JayanthT7 JayanthT7 commented Feb 14, 2026

Hi @hishamco , I’ve updated the README with LocalDB and standard SQL connection string examples to help new developers get set up faster. Please let me know if any changes or improvements are needed. I would be happy
to update the PR accordingly.

Thank you for your time and for maintaining this project.

Formatted based on existing font readme style
@JayanthT7
Copy link
Contributor Author

JayanthT7 commented Feb 15, 2026

Hi @hishamco , Thank you for the feedback. I had updated the README to explicitly mention creating the SimplCommerce database and how to adjust the connection string for different needs. Hopefully, this makes the setup process much clearer for new users!

@hishamco
Copy link
Member

Seems formatting JSON is annoying, @JayanthT7, please add the closing ``` for the JSON, then I will merge

@JayanthT7
Copy link
Contributor Author

Seems formatting JSON is annoying, @JayanthT7, please add the closing ``` for the JSON, then I will merge

Added the ending quotes for json.

@hishamco
Copy link
Member

Still there's formatting issues

@JayanthT7
Copy link
Contributor Author

Still there's formatting issues

Hi @hishamco , I've verified the raw Markdown to ensure all code blocks are properly terminated. The preview and the file diff now show the blocks are correctly fenced. Verified in preview of github as shown in attached image. Please let me know the exact space or line to be fixed if found breaking.

image

@hishamco hishamco merged commit 45e2248 into simplcommerce:master Feb 18, 2026
9 of 11 checks passed
@hishamco
Copy link
Member

Thanks

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.

3 participants