test post 2

cluse started this discussion on Oct 28, 2024 in test_forum3

Back to test_forum3

cluse
Catherine (u/cluse) Forum Admin
Posted 1 year ago
Edited 1 year ago

dihnetohitnoei

Comments (3)

[Deleted]
· posted 1 year ago

idkxhtbnm

idxhtb

[Deleted]
· posted 1 year ago

short to-do list

  • Enforce max length of comment

user profile page

  • bio has 500 character limit
  • you can edit your own profile, but not someone else's.

channel creation and settings page

  • Backend does not allow creating a username or channel name with a period or special characters other than underscores
  • as admin of channel, you can edit it, and not otherwise.
  • channel description has 500 character limit
  • fix bug - when channel is edited, all users are added as admins
  • when updating channel rules, make sure rules are an array of objects with summary and detail keys
  • if you create channel but don't pass auth rules, the channel should not be created
  • if you create a channel with a name that is already taken, you should see a human readable error in the UI (currently says constraint validation failed)
  • in discussion list view, clicking image should not jump to top of page in background

auth

  • fix error screen on logout

discussions

  • load more
    • load more button in phx_music does nothing
    • load more button shows where it should not - in sourceit channel page?
  • in forum list, it says phoenix music has 12 discussions, but in channel view, it says it has 8
  • channel create discussion form has console errors
  • edit discussion form should not say you don't have permission while form is loading
  • fix cache/console errors when editing discussion or loading discussion edit form

events

integration tests

  • get server config, default server role, and default channel role from the dev
    • add them to seed data
    • allow cypress permissions to do the drop data operations at the beginning of each test run
    • should delete mod profiles, comments, server roles, server config, and channel roles when integration tests are done.
  • use insomnia for API testing
    • collections
      • admin
      • standard user
      • suspended user
      • validation
  • graphql permissions.ts fully enabled and working with integration tests. Tests are in CI/CD. on the main branch, require tests to pass before merging

not coding

feedback

  • when doing the mutation addFeedbackCommentToComment (or adding any comment), reject it if no author name or modProfileName is provided
  • add a 15 character minimum length for feedback and reports
  • Backend and frontend should reject comments without channel names (currently it seems comments on events don't have channels attached)
  • reject feedback on comment, discussion or event if it doesn't say what it was giving feedback on
  • reject feedback on discussion if missing text or mod profile name
  • reject feedback on event if missing text or mod profile name
  • "edit your feedback" form looks weird at narrow screen width

style glitches

  • comment buttons not aligned if user is not logged in
  • fix side nav css - hover goes all the way across, add divider, fix padding
  • in sitewide discussion list, left side of expanded image should be more aligned
  • issue detail page should open at the top, not middle of the page

deletions

  • After deleting a discussion it should no longer be visible from the channel list. however,
  • the comments can still be seen on the authors' user profiles
  • if you go directly to the link where it used to be (which is the permalink on the comment on the user profiles), you can see the comment section
  • if a discussion or event is edited to disconnect, you can still see the discussion in a channel but the user and discussion body say they are deleted.
    • add test
  • if a discussion or event is disconnected and reconnected, the discussion or event in the channel should be restored, with the same comments visible as before. confirm that after removing and re-adding a discussion to a channel, the same comments are still there.
    • add test
  • should not see errors when you go to a permalinked comment on a discussion that has been deleted. it should just say the discussion could not be found http://localhost:5173/channels/[c/phx_music](https://www.topical.spacechannels/f/phx_music/discussions/discussions/d/45692279-2781-4d07-99ba-69094db9dd73/comments/6b9d76c1-edbb-48bb-8ddb-e4eb40097742
  • when a discussion is deleted, the corresponding discussion channels should either be deleted along with any comments, or the remaining discussion channels should still be rendered in the channel with the titles still remaining and the authors deleted. Right now there are a lot of orphan discussion channels (in the prototaxites channel, for example) for discussions that no longer exist. Those should either be displayed or get cleaned up.

test comment