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

Allow prefixing the syncing toots and tweets #50

Closed
wants to merge 4 commits into from

Conversation

pan93412
Copy link

@pan93412 pan93412 commented Nov 19, 2022

Fix #79

Description

Allow adding a prefix to the message, which is going to forward to Mastodon-side or Twitter-side. It is useful when you want to let readers know where this message is from.

image

Implementation

It will add prefix to every messages forward – it is implemented in send_single_post_to_(side).

@pan93412 pan93412 changed the title Allow prefixing the toots and tweets synced Allow prefixing the syncing toots and tweets Nov 19, 2022
@klausi
Copy link
Owner

klausi commented Jan 8, 2023

Thanks, I'm not sure yet if we should implement this new feature. It adds quite a bit of complexity to the code base.

Before we discuss implementation details of the pull request I would like to get some feedback, anyone else that needs this feature?

@nfrankel
Copy link

What I'd need is to remove the RT prefix from Twitter to Mastodon. It would be nice is it could be configured, e.g., RT as of now, nothing, Twitter, etc.

@klausi
Copy link
Owner

klausi commented Jun 4, 2023

@nfrankel I think that would be a different feature request for retweets/reblogs compared to the proposed changes here, please open a new issue for that.

@nfrankel
Copy link

nfrankel commented Jun 4, 2023

As per you request #79

@pan93412
Copy link
Author

pan93412 commented Jun 4, 2023

Added the Fix note in this PR description. BTW, I currently leave the PR as it is (I don't resolve conflict at this monent).

@pan93412 pan93412 closed this Jul 8, 2024
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

Successfully merging this pull request may close these issues.

Make hard-coded prefixes configurable
3 participants