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

Add labels when forwarding a link to bot #5

Open
vronizor opened this issue Mar 5, 2024 · 9 comments · Fixed by #6
Open

Add labels when forwarding a link to bot #5

vronizor opened this issue Mar 5, 2024 · 9 comments · Fixed by #6
Assignees
Labels
enhancement New feature or request review For issues where people leave review!
Milestone

Comments

@vronizor
Copy link

vronizor commented Mar 5, 2024

Hi! Thanks for the great bot :) I might finally be able to solve my information clutter by forwarding articles I want to read directly to my Omnivore!

I was wondering if it would be possible to add labels when forwarding? For example, it would be cool to have a "fixed" label saying that it came from Telegram (telegram), another one saying from which channel/contact it came (from_<contact>), and be able to add arbitrary ones at the time of forwarding (news or investigation).

I have no idea if any of that is possible, but just wanted to throw the idea out there.

@Rnbsov
Copy link
Owner

Rnbsov commented Mar 6, 2024

Hello! I'm thrilled to hear that it was helpful for you ;)

  • The fixed/default label seems to be easy to implement, that's a great idea!

  • Speaking of the source from which article was forwarded from is also simple, but I'm not quite sure if this possible, gotta figure it out

I was already thinking of feature to be able to add labels for articles, basically I see it so:

you're sending link and then enumerate labels for it like link label1 label2 label3

Personally, I have label consisting from two words or including emojis 👾🌌🍊, to make it easier to avoid misspelling or spend time to search for certain symbol I wanted also add labels for labels feature (tautology😆), like you could add cs label to write it instead of computer science each time

@Rnbsov Rnbsov added enhancement New feature or request review For issues where people leave review! labels Mar 6, 2024
@Rnbsov Rnbsov self-assigned this Mar 6, 2024
@Rnbsov Rnbsov linked a pull request Mar 9, 2024 that will close this issue
@Rnbsov Rnbsov closed this as completed in #6 Mar 9, 2024
@Rnbsov Rnbsov reopened this Mar 9, 2024
@Rnbsov Rnbsov added this to the Suggestions milestone Mar 9, 2024
@vronizor
Copy link
Author

vronizor commented Mar 9, 2024

That sounds perfect! Thanks for the hard work 👍

@Rnbsov
Copy link
Owner

Rnbsov commented Mar 10, 2024

Always happy to help!

The first two requested features were implemented under /set_default_label and /set_include_source commands 😸👍

Regarding the ability to add tags while saving a link , it's now possible! but currently it's not possible to save labels consisting from two words (they will be interpreted as two separate), although I've come up with idea to add some kind of enclosure for them maybe using single/double quotes like link "ai breakthroughs" 'computer science'

As for labels for labels is also not implemented just yet

Feel free to reach out if you have further questions or suggestions! 👾

@vronizor
Copy link
Author

I don't know how hard that would be, but it would be great to have the ability to add labels while forwarding a link/message with a link from a channel.

Again taking the example from HackerNews Feed, it would be super useful to hit the forward button and add a message containing the labels. Telegram calls that " Add a comment" in the forwarding menu. See the screenshots below for an illustration.

Screenshot_2024-03-13-07-51-52-663-edit_org telegram messenger

Screenshot_2024-03-13-07-49-47-027-edit_org telegram messenger

@Rnbsov
Copy link
Owner

Rnbsov commented Mar 20, 2024

@vronizor Happy to inform you that's now possible! 😸👍

@vronizor
Copy link
Author

Thanks @Rnbsov ! will test asap

@vronizor
Copy link
Author

Hi! I've been testing on my end and it doesn't seem to work so far. Here, the "geo" tag should have been added but it doesn't show up on omnivore.

Screenshot_2024-03-24-13-07-45-064_org telegram messenger
Screenshot_2024-03-24-13-08-07-678-edit_app omnivore omnivore

@Rnbsov
Copy link
Owner

Rnbsov commented Mar 25, 2024

ugh, should work fine now

@vronizor
Copy link
Author

Super, works !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request review For issues where people leave review!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants