Skip to content

feat: replace twitter links with vxtwitter alternative #5

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 5 commits into from
Jul 18, 2023
Merged

Conversation

potb
Copy link
Contributor

@potb potb commented May 19, 2023

No description provided.

Copy link
Contributor

@ayoubqrt ayoubqrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi thanks for this feature it will be useful but I am doubtful about the appearance of the bot when someone will send a twitter link. But let's test and we will see if it's okay 👍 .

return;
}

const urls = message.content.match(/https?:\/\/\S+/g);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi, can you please make a check if there is some twitter links in the message ? Because the bot will send a message on every message.

'https://vxtwitter.com'
);

const newMessage = [`<@${message.author.id}>`, newContent].join('\n');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if it's clear enough for the users to only mention the name of the user. It can be better to put Message de : @Ayoub or something else 🤷‍♂️

@potb potb closed this Jul 15, 2023
@potb potb deleted the vxtwitter branch July 15, 2023 02:06
@potb potb merged commit 98dcf28 into master Jul 18, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants