Poprawki do PR-a #4

[ScriptyChris]

Ze względu na brak reakcji @CodeKid0 na sugestie poprawek do PR-a #4 i brak dostępu do jego repozytorium, skąd pochodzą zmiany, postanowiłem dodać je na osobnym branchu.

[d-bend]

Moim zdaniem można mergować tak jak jest.,

[awaluk]

Moim zdaniem nie za bardzo działa funkcjonalnie :P Gdy osoba, która nie ma uprawnień do everyone wysyła jakąś wiadomość, to bot się wywala

discord-bot/handlers/mentionHandler.js:20
const hasPermission = message.member.hasPermission(MENTION_PERMISSION);
^
TypeError: Cannot read property 'hasPermission' of null

Co ciekawe gdy wyśle to osoba, która może użyć everyone, działa ok (czyli nic się nie dzieje, ale też nie rzuca błędu). Do tego embed z informacją trafia do piszącego na priv zamiast na kanał

[ghost]

Ze względu na brak reakcji @CodeKid0 na sugestie poprawek do PR-a #4 i brak dostępu do jego repozytorium, skąd pochodzą zmiany, postanowiłem dodać je na osobnym branchu.

Przepraszam, ale aktualnie na wakacjach 😅

[ghost]

Moim zdaniem nie za bardzo działa funkcjonalnie :P Gdy osoba, która nie ma uprawnień do everyone wysyła jakąś wiadomość, to bot się wywala

discord-bot/handlers/mentionHandler.js:20
const hasPermission = message.member.hasPermission(MENTION_PERMISSION);
^
TypeError: Cannot read property 'hasPermission' of null

Co ciekawe gdy wyśle to osoba, która może użyć everyone, działa ok (czyli nic się nie dzieje, ale też nie rzuca błędu). Do tego embed z informacją trafia do piszącego na priv zamiast na kanał

W moim PR'u ten problem był naprawiony

[ScriptyChris]

W moim PR'u ten problem był naprawiony

W Twoim kodzie był try..catch, który łapał ten błąd. Pytanie, czy ten błąd pojawiał się zawsze, czy w losowych przypadkach? Jeśli zawsze, to może warto inaczej sprawdzać uprawnienia usera, żeby taki błąd się nie pojawiał, niż polegać na catchu.

P.S. Wyżej usunąłem zduplikowany komentarz od CodeKid0.