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

Topic Archival #9871

Merged
merged 22 commits into from
Nov 13, 2024
Merged

Topic Archival #9871

merged 22 commits into from
Nov 13, 2024

Conversation

timolegros
Copy link
Collaborator

@timolegros timolegros commented Nov 9, 2024

Link to Issue

Closes: #9825

Description of Changes

  • Switched from 'delete topic' to 'archive topic'
  • Added topic unarchival

Test Plan

Deployment Plan

#9825 (comment)

For the 3 threads in the common community, someone from growth should assign them a topic and for the 2 in dhk we should reach out to them or assign a random topic:

Other Considerations

@timolegros
Copy link
Collaborator Author

@mzparacha, the backend for this PR is complete. When you hit 'delete topic' on the client, it will archive the topic and its threads (they show up in archived). What remains to do on the client:

  • Update button/modal warning text to reflect archival
  • Add 'unarchive' functionality -> This may not be necessary for this PR (maybe a separate ticket?)
  • Verify there are no unintended side effects from loading topics anywhere else

@mzparacha
Copy link
Contributor

@timolegros I did this frontend work

  • Archiving/Unarchiving topics + refreshing thread cache when this happens
  • Testing for side effects (didn't find any)

I did find 1 issue with unarchiving a topic -> when unarchived the threads of that topic remain archived for some reason, could you please take a look a this. Also, I couldn't verify cache updates for this case in frontend, will try it after we update the command.

@timolegros
Copy link
Collaborator Author

@mzparacha fixed the bug. For caching on the client most everything seemed to work except 1 thing: after archiving a topic the archived threads did not show up in the archived section until I refreshed. Other than that I think this is ready.

# Conflicts:
#	libs/model/src/community/DeleteTopic.command.ts
@timolegros timolegros added the deployment plan (PRs only) requires manual infrastructure changes on release label Nov 13, 2024
@mzparacha
Copy link
Contributor

@mzparacha fixed the bug. For caching on the client most everything seemed to work except 1 thing: after archiving a topic the archived threads did not show up in the archived section until I refreshed. Other than that I think this is ready.

Tested but wasn't able to repro

Screen.Recording.2024-11-13.at.5.44.25.PM.mov

cc: @timolegros

@timolegros
Copy link
Collaborator Author

@mzparacha fixed the bug. For caching on the client most everything seemed to work except 1 thing: after archiving a topic the archived threads did not show up in the archived section until I refreshed. Other than that I think this is ready.

Tested but wasn't able to repro
Screen.Recording.2024-11-13.at.5.44.25.PM.mov

cc: @timolegros

huh ok no worries then. Can you push a fix for the lint errors and then I will put this up for review?

@timolegros timolegros marked this pull request as ready for review November 13, 2024 18:49
# Conflicts:
#	libs/model/test/community/community-lifecycle.spec.ts
#	packages/commonwealth/client/scripts/views/pages/overview/index.tsx
@ilijabojanovic ilijabojanovic merged commit db67171 into master Nov 13, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment plan (PRs only) requires manual infrastructure changes on release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Discussion page crashes on some communities
5 participants