Skip to content

Replying to mentions#65

Closed
hfgd123 wants to merge 7 commits intoScootKit:mainfrom
hfgd123:react
Closed

Replying to mentions#65
hfgd123 wants to merge 7 commits intoScootKit:mainfrom
hfgd123:react

Conversation

@hfgd123
Copy link
Copy Markdown
Contributor

@hfgd123 hfgd123 commented May 29, 2022

As suggested by shini in https://features.sc-network.net/board/custombot/412 I added a reply-to-mention feature in the auto-react-module. Something went wrong with my Webstorm branches again, so the commits are made directly on the GitHub website, pasted from webstorm

@SCDerox
Copy link
Copy Markdown
Member

SCDerox commented May 29, 2022

Please add support for embeds using config elements

Comment thread modules/auto-react/config.json Outdated
@hfgd123 hfgd123 requested a review from SCDerox May 30, 2022 18:40
Comment thread modules/auto-react/events/messageCreate.js Outdated
@hfgd123 hfgd123 requested a review from SCDerox May 30, 2022 19:52
Copy link
Copy Markdown
Member

@SCDerox SCDerox left a comment

Choose a reason for hiding this comment

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

Sorry, but this does not work at all - several issues such as comparing objects with IDs, wrong looping and others need to be addressed...
I will fix these issues (I spent way too much time trying to give you feedback, but at this point it's easier to do it myself completely, sorry).
You can find a ZIP of the working version attached.
auto-react.zip

@hfgd123 hfgd123 closed this May 31, 2022
@hfgd123 hfgd123 deleted the react branch May 31, 2022 11:27
@hfgd123
Copy link
Copy Markdown
Contributor Author

hfgd123 commented May 31, 2022

I shouldn't do anyting while I'm tired

@SCDerox
Copy link
Copy Markdown
Member

SCDerox commented May 31, 2022

No worries, I know this too well, if I am tired I can't do anything right ^^

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.

2 participants