Skip to content
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

Issues with Announcement Messages #9741

Open
RockRoller01 opened this issue Jan 14, 2023 · 0 comments
Open

Issues with Announcement Messages #9741

RockRoller01 opened this issue Jan 14, 2023 · 0 comments

Comments

@RockRoller01
Copy link
Contributor

RockRoller01 commented Jan 14, 2023

Returning to using announcement messages after a while, the issue in #9002 still persists, and I have noticed a couple new ones

  • descriptions feel useless. in my experience they are just a hassle to fill out and a duplicate of the title (Make announcement channel description optional #9770)
  • the GUI does not show the limits for lengths. These are title (50), description (255), message (1024) (Add message length counter to create announcements UI #9752)
  • the API docs do not mention these limits either
  • API docs have a mistake, they say that target_ids is required if PM, else ignored. In reality its required for ANNOUNCE, ignored for others. (fixed, not sure which issue)
  • descriptions dont wrap if there is no spaces (very minor thing, noticed this while testing with random inputs, but i can totally see someone just holding down - because descriptions are annoying, maybe relevant on mobile with a very long word..?)(Always wrap chat announcement description #10522)
  • there seems to be no check besides the GUI for title and description length, trying to send something longer via api (or console, or modify maxlength) results in 500 error
  • the editor not filling the full screen height and just ending somewhere in the middle feels weird (Larger form for creating announcements #9753)

image

image

Overall a pretty long list of things that should be improved with announcement messages, many of them actually being relevant and not just "oh, you will find these if you do intense testing for edge cases" things. I can split this list into a bunch of seperate issues if wished for.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Next up
Development

No branches or pull requests

2 participants