-
Notifications
You must be signed in to change notification settings - Fork 0
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
PB-58: Bot dm's user instead of joining general #88
Conversation
e0adaef
to
2b75bd6
Compare
Usikker på om jeg også skal fikse hva bot-en sier i velkommen meldingen, men det kan kanskje designerene være med på å bestemme? |
…r None old channel_id
…r authed user field
c722306
to
26b7a2c
Compare
Enig at det kan ventes med, bra jobba! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, bra jobba 🚀
Kitemaker Ticket
Gjør egt akk det som kitemaker beskriver, men tror jeg venter med å refakturere velkommenmeldingen.
Tror jeg har handlet de viktigste edge-casene, men med en safeguard på events slik at vi kan i allefall oppdage feil senere uten at noe breaker massivt.