search

Together-100Devs / Together

Together is a group calendar application using the MERN stack intended to bring discord communities closer!
https://together.cyclic.app/
MIT License
167 stars 112 forks source link

Adding event with more than 280 chars sends 400 error on /events without UI feedback #417

Closed nmpereira closed 1 year ago

nmpereira commented 1 year ago

Describe the bug

Adding event with description text longer than 280 chars sends a 400 error on /events post, but UI doesn't show any error

To Reproduce

Steps to reproduce the behavior:

  1. Go to https://together.cyclic.app/
  2. Add event after logging in add a description with more than 280 characters
  3. Add all other event details as usual (added non recurring - 2023-05-15 6.30pm to 7.30pm EST)
  4. Submit event

Expected behavior

Event is added

Actual behavior

Modal says event is added successfully but debugging showed that the /events failed with a 400 error 

"\"description\" length must be less than or equal to 280 characters long"

Screenshots

chrome_xDfzAYiVSV

Which Branch should this fix be Pulled into?

PR into [branchName]

Device Information

Desktop (please complete the following information):

Additional context

Possible solutions:

  1. remove the 280 char limit on the endpoint
  2. show user feedback that the event has not been added when there is a failed response
  3. The HTML text box should limit the user from adding more than 280 chars
Caleb-Cohen commented 1 year ago

Thank ya for pointing this out!