SNEWS2 / SNEWS_Publishing_Tools

Publishing Tool for SNEWS
BSD 3-Clause "New" or "Revised" License
2 stars 2 forks source link

publishing messages #4

Closed KaraMelih closed 2 years ago

KaraMelih commented 2 years ago

I just pushed an updated version of the CLI. I'd like to raise two issues;

First; Right now publish method does not allow extra arguments. While this might be the desired use. I think it should not fail to publish, rather

Second, I think the snews_pt is meant for users to interact with the system without having to deal with any backend tools. The publishing tools are already nice and usable, however, we should also carry the means to subscribe to alert channel, and I think this subscription should also allow downloading the received alerts in json or some other format. I don't think users would monitor the terminal all the time.

I also would like to suggest starting using python config files for broker, channel, token, etc (basically replacing the env file). This way later, we can either ask users to put their experiments once in the config file or try to fetch it from hop.auth. I can work on that if agreed.

KaraMelih commented 2 years ago

About allowing extra arguments without crashing we can refer to this

KaraMelih commented 2 years ago

fixed.