discord-webhooks icon indicating copy to clipboard operation
discord-webhooks copied to clipboard

Add error checking to various parts

Open nopjmp opened this issue 6 years ago • 2 comments

To prevent human errors and throwing back sometimes obscure cURL error messages, we should check the parameters to make sure they are valid.

  • [ ] URL check with filter api
  • [ ] Message length checks
  • [ ] Embed validation
  • [ ] Find more places for validation

nopjmp avatar May 15 '18 13:05 nopjmp

@nopjmp feel free to check my fork of your rep https://github.com/JohnMagdy/discord-communicator (the project is temporary paused)

Zorono avatar May 19 '18 23:05 Zorono

It's a little difficult to parse out change changes since you keep deleting and uploading through GitHub instead of using git locally when you make changes.

Quite a few additions I see as anti-features in the aspect of static strings of what color is what. It will also be hard for you to merge my code base back into your's if I do update.

Thanks for mentioning your repository though!

nopjmp avatar May 19 '18 23:05 nopjmp