search

ubiquibot / conversation-rewards

0 stars 14 forks source link

Rename the `listeners` item to `watch` #60

Open gentlementlegen opened 1 month ago

gentlementlegen commented 1 month ago 
          Sorry to keep nitpicking. I just want it to be clear as possible for new developers. We can always rename these later later in the future so it's by no means a priority or urgent to address this comment.

What about "ubiquity:watch"?

Given that we are conforming to the manifest standard, I wonder if it's realistic to repackage chrome extensions into plugins.

If we can set the event to watch on the bot config side, then in theory we don't need to make any adjustments to the chrome extension code/manifest to interoperate.

It could be useful to tap into a mature ecosystem of capabilities.

_Originally posted by @0x4007 in https://github.com/ubiquibot/conversation-rewards/pull/59#discussion_r1678835108_

0x4007 commented 1 month ago

I think let's hold off on this one for now. We can finalize the terminology after shipping v2