• Mark images as high DPI in report room
• Generate HTML that scales the images correctly

My idea was to tag screenshots with both, the notice ⭕ and something like 2️⃣ for 2x scaling. Would require enhancing the data structure for News.images. Would probably make sense to make it a struct at this point?

This is my first attempt to do anything serious with Rust, and I suspect I'm breaking a lot of rules. However, it appears to work, so hopefully we can tidy it up. Examples:

Reporting room:

Gwmngilfen: newsbot 🗞️: this report is using a display name!
newsbot 🗞️: ✅ Thanks for the report Gwmngilfen, I'll store your update!
Gwmngilfen: @hebbot:mydomain.org this report is using a full id
newsbot 🗞️: ✅ Thanks for the report Gwmngilfen, I'll store your update!
Gwmngilfen: hebbot this report is using a localpart only
newsbot 🗞️: ✅ Thanks for the report Gwmngilfen, I'll store your update!

Rendered MD:

[Gwmngilfen](https://matrix.to/#/@gwmngilfen:ansible.im) says
> this report is using a full id

[Gwmngilfen](https://matrix.to/#/@gwmngilfen:ansible.im) says
> this report is using a localpart only

[Gwmngilfen](https://matrix.to/#/@gwmngilfen:ansible.im) reports
> this report is using a display name!

The code is really ugly. In particular, I think I should just pass the whole Bot object to msg_starts_with_mention and then extract the user_id and displayname inside - but I couldn't work out how to make the type-checking work with that. Instead I'm left with a lot of string unwrapping that is fairly nasty.

Also, I see there's a nice method to get display names for reporters - but that requires a RoomMember object, which the bot is not - could that be reused in some way?

Looking forward to the feedback!

Sometimes people can forget to mention the bot when reporting news. It's quite noisy to copy the former message, potentially redact it and send a new one by mentioning the bot first.

If the person who reported the news could react to it with an emoji (⭕ by default, configurable?) to tell the bot it actually was a news report, it would make reporting rooms less noisy. Bonus points: it's consistent with twim-o-matic 😇

Then the editor can approve the news e.g. with the ✅ emoji (instead of ⭕ by default today)

It would be very helpful to be able to set:

• How section are rendered (variable available: {{section.emoji}}, {{section.title}})
• How projects are rendered (variables available: {{project.emoji}}, {{project.title}}, {{project.website}}, {{project.description}})

It would typically allow people to have sections that are h2 title in markdown and not h1 title by declaring them as ## {{section.title}} and to change how project titles are rendered, e.g. as ### {{project.title}} ([link]({{project.website}}))

When the bot display starts the same as the MXID, but has extra characters, then the truncation goes wrong. I noticed this because my bot is on my Conduit server, and Conduit adds :zap: too all display names by default :)

ID: "hebbot@mydomain" Display: "Hebbot :zap:"

News item: "hebbot :zap:: some more news is you! A thing is good!" Rendered text: "> ⚡️: some more news is you! A thing is good!"

You can see it has left the :zap: and the colon behind.

OK, this was a bit harder, I learned some Rust :)

I'm not sure if the reading of the template files is in the right place, but I also wasn't sure about how to pass the resulting strings around, so this seemed cleaner. Feedback very welcome!

My first attempt at Rust, so comments very welcome :)

Also, for some reason, using "./config.toml" etc wasn't working in gitignore while I was testing, so I changed "./" to just "/" so it ignores those files in the project root. Can drop that commit if it's a problem, just let me know.

This relates to this news item

So it would seem that using the reaction to flag a post overwrites any previous metadata. This is the timeline:

1. The above item is posted with a correct mention of the bot, and the bot notices and replies
2. Editor adds a project reaction (🪜)
3. Editor adds a submission reaction (:o:)

Result: no project/section (via default_section) is stored:

• ℹ️ 1 news are not included because of project/section assignment is missing. Use !status command to list them. ❌ Unassigned / ignored news entries (1):
• [open message] @andersson007_:matrix.org: the [community.libvirt](https://github.com/ansible …

Looking at the logs around the submission, I see this:

✅ @andersson007_:matrix.org submitted a news entry. [open message] ✅ Editor @cybette:ansible.im added the project description “Maintainers” to @andersson007_:matrix.org’s news entry [open message]. ✅ @andersson007_:matrix.org submitted a news entry. [open message]

The second "submission" is clearly a duplicate caused by step 3 in the timeline. It should:

1. Not overwrite the previous submission
2. Probably log something different so editors know that it is unneccessary

Hebbot is already able to distinguish between a picture and a video. It shouldn't be necessary for the editor to make the distinction.

Simply using 📷️ for both should work.

When a news entry gets announced in the admin room as follows:

✅ @person:provider.org submitted a news entry. [open message]

Replying to that message with project projectName should:

• Make the bot add the reaction emoji corresponding to the project in the news report
• Add the news entry in the corresponding section

Replying to the same message with clear project should make the bot un-react to the original report and remove it from the corresponding section.

Hebbot adds a :heavy_check_mark: reaction to commands in the admin room when it successfully executed it Hebbot adds a :x: reaction to commands in the admin room when it failed to execute it

Atm the names of the files downloaded with the curl command are quite cryptic. Templatifying the names of the downloads would be great. The following variables make sense imho:

• yyyy year in 4 digits format
• mm month of the year
• dd day of the month
• section name of the section in which the related news is in (if any, nosection if in none), kebab cased
• project name of the project to which the related news is in (if any, noproject if in none), kebab cased

In case of duplicate name, appending a -n with n being an integer should do the trick

An example of template for the downloads could be: {{ yyyy }}-{{ mm }-{{ dd }}-twim-{{ section }}-{{ project }} The resulting downloaded file would be:

• 2022-10-14-twim-clients-element-ios.png
• 2022-10-14-twim-clients-element-ios-1.png
• 2022-10-14-twim-servers-noproject.jpg

When #69 is merged, it would be very handy to be able to ask hebbot to scan a message again and post reactions to the message with the matching projects.

Since every section and project seems to have a sort of id ("name") in addition to its title, is this not an obvious place to use an object instead and change

[[section]]
name = "foo"
title = "Foo"

to

[section.foo]
title = "Foo"

?

In some circumstances it is nice to have some indication that the bot saw your report via reactions - especially where there is heavy use of notice_emoji which suppresses the text acknowledgement.

This PR adds an ack_emoji, and allows for it to be disbled in the config. Best paired with #62 which would allow you to either disabled all text and just use reactions, or disable reactions and just use text, or a mix of both.