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

Thread Slack posts #38

Open
zachlipton opened this issue Apr 3, 2020 · 0 comments
Open

Thread Slack posts #38

zachlipton opened this issue Apr 3, 2020 · 0 comments

Comments

@zachlipton
Copy link
Collaborator

The #urlwatch Slack channel is often unreadable because of the volume and format of the bot's posts. Totally-low-priority convenience feature only worth spending time on if you want a diversionary activity, but it would be kind of nice if it printed the summary table at the top and then posted one Slack thread per state, something like:

CHANGED: Alabama ( https://services7.arcgis.com/4RQmZZ0yaZkGR1zy/arcgis/rest/services/COV19_Public_Dashboard_ReadOnly/FeatureServer/0/query?where=CONFIRMED>0&outStatistics=[{"statisticType":"sum","onStatisticField":"CONFIRMED"}]&cacheHint=true )
1 reply

And then the actual diff would be in the "1 reply."

This would require modifying SlackReporter, which currently assumes its getting a big block of text from its superclass and spitting it out. The WebhookReporter that follows it can serve as an example of getting the actual data to format it. It will also require some Slack API work to get the ts value of the parent message and supply that when posting the child.

If you need Slack keys or permissions to test, just let me know.

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

1 participant