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

Support Connect / Disconnect commands via msg #107

Draft
wants to merge 4 commits into
base: next
Choose a base branch
from

Conversation

Steve-Mcl
Copy link
Collaborator

Supports user "connect" and "disconnect" via msg - to permit runtime recovery in the event of error.

This is ground work for supporting runtime settable connection parameters at a later time.

Features:

  • accepts topic : "command" + payload: "connect" or payload: "disconnect"
  • updates all nodes status via a local event bus
  • ensures pool is destroyed
  • Updated en-US and zh-TW node built in help

Unit tests expanded - now has:

    ✔ should be loaded (876ms)
    ✔ should connect to database when topic and payload are set (49ms)
    ✔ should disconnect from database when topic and payload are set (1528ms)
    ✔ should perform a simple query (44ms)
    ✔ should return data to specified property
    ✔ should perform a simple using ui configured query with mustache (52ms)
    ✔ should can create table and insert/select data

Co-authored-by: Shao Yu-Lung (Allen) <[email protected]>
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.

2 participants