Skip to content
This repository was archived by the owner on Mar 17, 2025. It is now read-only.

Conversation

@ed7coyne
Copy link
Collaborator

No description provided.


##BEGIN
Must be called after creating a Serial connection, it can take either just a host for accessing public variables or you may also provide a secret for accessing protected variables in the database.
##BEGIN_DB
Copy link
Contributor

@proppy proppy Aug 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what do you think of doing BEGIN SERVICE ARGS?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm... I am not sure. I could see some advantages. It may make the service registration system easier. You can try to load a service called "SERVICE" when the command is actually called.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's think about it some more. I am not going to block on it. We will need to change the database implementation to match so it could be a followup PR.

>> MSG_USERS fQCLfBOGdh0...9k0 fQCLfBOGdh0...5j1
<< +OK

## MSG_TOPIC
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe specific which parameter is optional and if there is a default.

time_to_live=500

##Usage
MSG %Argument% %Argument% ...
Copy link
Contributor

@proppy proppy Aug 26, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you think a single line variant would be useful? (maybe we can distinguish with suffix like we did originally for db).

Something like:
MSG "TITLE" "INLINE_BODY" [data_key=data_value ...]

@proppy
Copy link
Contributor

proppy commented Sep 26, 2016

Let's merge and iterate!

@proppy proppy merged commit a43920c into FirebaseExtended:master Sep 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants