Sprint#4 feedback

[tien-han]

Team feedback:

1. Team Ant
   1. If a user is allowed to enter a URL, make sure to validate it
   2. Make sure columns aren't too squished to each other
2. Team Gnocchi
   1. Make sure that there is confirmation that a user has been made an admin
   2. Make sure a person can't remove themselves as an admin (case: what if the only admin removes themselves as an admin?)
3. Team Earwig
   1. Announcements - make sure more of the title is legible so that people know what they're for
4. Team Dragonfly
   1. They have a include deleted users button on their tables (that could be good to have to show soft deleted
      users)
   2. Maybe don't edit an announcement (you can just email + have a receipt page) -> just delete it at the moment
   3. Make sure that there's form validation on every browser (that you expect)
   4. Make sure to prevent user from adding in PHP tags that can break features
5. ✨ Team Cicada ✨
   1. New applications - remove requirement for contact info
   2. For sprint # 5 views
      1. Users should only see user views
      2. Admin should only see admin views
      3. Users & admins can see both views
   3. Applications table - make sure to remove the time for the date column
   4. Fix the "squished" styling for the Update and Delete buttons
   5. There should be validation to ensure that users can't have any role (PR is out already)
6. Team Beetle
   1. For announcements, allow the admin to make an announcement to all cohorts
   2. Give breathing room for the table columns
   3. Don't get caught scrolling in a table (that is spread out horizontally) - don't enable horizontal scrolling

Sprint # 5 Considerations:

• Login that stores user password not in plain text (hash and salt passwords)
• Might be good to let admin see the user side (in case the admin needs to troubleshoot what happens with a user)
• Check what the website looks like on mobile