Skip to content

Fixed devy role issue#60

Merged
adavila0703 merged 2 commits intomainfrom
devyrole-bug
Dec 12, 2021
Merged

Fixed devy role issue#60
adavila0703 merged 2 commits intomainfrom
devyrole-bug

Conversation

@adavila0703
Copy link
Copy Markdown
Collaborator

There is something weird with how we are getting a "guild" and "roles" from a member. We can grab the guild_id and the member_roles from the incoming message.

  • Removed guildFromStateF and memberFromStateF
  • Getting guild id and member rolls from message
  • Confirmed that this fixed the previous bug with the devy bot.

Close #56

Comment thread internal/devhubbot/commands.go
Copy link
Copy Markdown
Collaborator

@dillonstreator dillonstreator left a comment

Choose a reason for hiding this comment

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

LGTM

@adavila0703 adavila0703 merged commit 40440f0 into main Dec 12, 2021
@adavila0703 adavila0703 deleted the devyrole-bug branch December 12, 2021 14:22
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.

Bug: !devydeveloper only removing and not adding role

2 participants