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

addNotice in controller class doesn't match API used by local python interface #6

Open
bpaauwe opened this issue Feb 7, 2020 · 1 comment

Comments

@bpaauwe
Copy link

bpaauwe commented Feb 7, 2020

The addNotice API in the cloud interface is:
def addNotice(self, data):
while in the local interface it is:
def addNotice(self, data, key=None):

Code written to make use of the 'key' parameter that works correctly with local polygot fails when running on cloud polyglot

@bpaauwe bpaauwe changed the title addNotice in controller class doesn't match API used by local python template addNotice in controller class doesn't match API used by local python interface Feb 7, 2020
@Goose66
Copy link

Goose66 commented Aug 1, 2020

Looks like currently the polyinterface addNotice() will handle a string message (value) with an optional key, a dictionary with a key and value (msg), or a list of dictionary objects. The pgc_interface addNotice() errors out on anything other than a dict.

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

No branches or pull requests

2 participants