Skip to content

feat(Mass Privater): Add mass unprivate functionality#1639

Open
marcustyphoon wants to merge 26 commits into
AprilSylph:masterfrom
marcustyphoon:mass-unprivater
Open

feat(Mass Privater): Add mass unprivate functionality#1639
marcustyphoon wants to merge 26 commits into
AprilSylph:masterfrom
marcustyphoon:mass-unprivater

Conversation

@marcustyphoon

@marcustyphoon marcustyphoon commented Oct 28, 2024

Copy link
Copy Markdown
Collaborator

Description

Per a number of user requests, this adds the ability to unprivate posts to Mass Privater, including by tag. There's currently no realistic way to do this without an extension, since you can't find all of your private posts (that contain a certain tag or otherwise) through the Tumblr UI. No, it's not fast, but it's faster than manually scrolling through your archive, that's for sure.

Resolves #1626.

Currently contains the diff of #1610; must be rebased after it is merged (or close it if merged first).

Testing steps

Oh boy. Um.

  • Confirm that Mass Privater's "private posts" mode still works as it used to.
  • Confirm that Mass Privater's "unprivate posts" mode functions (slowly, at one post per second)...
    • in general, with no errored post list if there are no errors
    • with a before date selected
    • with tags selected
  • Confirm that Mass Privater's "unprivate posts" mode displays a list of errored posts if some of its posts are not npf-editable (populated up-front) or fail (e.g. disconnecting network)

Comment thread src/features/mass_privater.js Outdated
Comment on lines +267 to +272
apiFetch(`/v2/blog/${uuid}/posts/${postData.id}`, {
method: 'PUT',
body: {
...createEditRequestBody(postData),
state: 'published'
}

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmmm. the fact that this will only work on NPF/text/answer posts is... concerning.

i feel like we should have a separate "incompatible" count to show to the user, rather than trying to action on them here, knowing that the operation will always fail.

do we have a util for NPF compatibility yet? i feel like we need one.
if a post is stored in blocks format, or the original type is text, or the original type is note, it should be editable via NPF endpoints.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, shoot.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(And yes we do; it currently returns isBlocksPostFormat || shouldOpenInLegacy === false;).

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made something. A bit rudimentary, maybe, but I'm not sure how to improve it.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it should be a <table>...?

@marcustyphoon marcustyphoon self-assigned this Nov 12, 2024
@marcustyphoon
marcustyphoon marked this pull request as draft November 12, 2024 22:41
@marcustyphoon marcustyphoon added the help wanted Extra attention is needed label Jan 4, 2025
@marcustyphoon marcustyphoon removed their assignment Jan 4, 2025
@marcustyphoon
marcustyphoon marked this pull request as ready for review February 7, 2025 14:02
@marcustyphoon marcustyphoon removed the help wanted Extra attention is needed label Feb 7, 2025
@marcustyphoon marcustyphoon self-assigned this Feb 7, 2025
@marcustyphoon
marcustyphoon removed the request for review from AprilSylph February 7, 2025 17:26
@marcustyphoon marcustyphoon removed their assignment Mar 17, 2025
@marcustyphoon

Copy link
Copy Markdown
Collaborator Author

Today I learned: I have absolutely no idea how to control overflow on a table cell.

@marcustyphoon marcustyphoon added the help wanted Extra attention is needed label Mar 17, 2025
@marcustyphoon marcustyphoon changed the title Mass Privater: Add mass unprivate functionality feat(Mass Privater): Add mass unprivate functionality Oct 25, 2025
@AprilSylph
AprilSylph self-requested a review October 28, 2025 11:42
@AprilSylph
AprilSylph removed their request for review February 6, 2026 16:56
@AprilSylph

Copy link
Copy Markdown
Owner

(Don't know why I put this in my review queue when it's still marked as "help wanted".)

@marcustyphoon

marcustyphoon commented Feb 6, 2026

Copy link
Copy Markdown
Collaborator Author

Oh, this one.

I might just give up on table and switch to flexbox grid; I don't think I've ever gotten a table to do anything close to what I wanted ever.

marcustyphoon added a commit that referenced this pull request Mar 3, 2026
@marcustyphoon

marcustyphoon commented Mar 3, 2026

Copy link
Copy Markdown
Collaborator Author

¯\_(ツ)_/¯

(aside: I guess this could also just be a list of links, with some placeholder text added for the posts without a summary. or... yeah, idk. hm, or should the links be formatted timestamps instead of post ids?)

@marcustyphoon

Copy link
Copy Markdown
Collaborator Author

maybe? idk

@marcustyphoon

marcustyphoon commented Mar 13, 2026

Copy link
Copy Markdown
Collaborator Author

Current look. Suggestions welcome.

@marcustyphoon marcustyphoon removed the help wanted Extra attention is needed label Mar 13, 2026
@AprilSylph

Copy link
Copy Markdown
Owner

Hmm. You know what, what's wrong with just a scrollable list of clickable post URLs? If the post has a summary, it should have a slug too...

@marcustyphoon marcustyphoon self-assigned this Mar 20, 2026
@marcustyphoon marcustyphoon added the help wanted Extra attention is needed label May 8, 2026
@marcustyphoon marcustyphoon removed their assignment May 17, 2026
@AprilSylph
AprilSylph removed their request for review May 26, 2026 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mass Unprivater as an option to Mass Privater

2 participants