search

snu-quiqcl / qiwis

QuIqcl Widget Integration Software
MIT License
5 stars 2 forks source link

Fix a duplicated called bug in pollor #124

Closed BECATRUE closed 1 year ago

BECATRUE commented 1 year ago

This PR is related to #82, not close.

Before fix all bugs, first of all I modified updateDB() in pollor.

Situation

  1. Add a database in dbmgr.
  2. Set the database as a store of poller.
  3. Remove the database from dbmgr.

Original

image

We can check that the polling message is duplicated.

Fixed

image

The duplicated message is removed.

BECATRUE commented 1 year ago

After your review, a few codes were modified, so I just made a PR.

@kangz12345

kangz12345 commented 1 year ago

Thank you for the screenshots!

BECATRUE commented 1 year ago

I updated the docstring. Please check it.

BECATRUE commented 1 year ago

I applied your reviews. @kangz12345