Skip to content

Conversation

@Oaphi
Copy link
Member

@Oaphi Oaphi commented Jul 23, 2025

DO NOT MERGE UNTIL #1513 IS IN - it's NOT targeted to develop at this moment

This PR addresses an issue with attempting to click on the "Create Post" button in categories without any allowed post types (usually happens with brand new categories). Currently, we incorrectly handle this case, redirecting the user to the page that asks the user to select a post type as the category has more than one (where in fact is has none):

Screenshot from 2025-07-23 05-15-55

This PR disables the abovementioned button and adds a corresponding title text (not shown in the screenshot, a bit hard to capture) for users who cannot edit allowed category post types (anyone other than an admin ATTOW). Due to category headers having configurable colors and the styling being managed by Co-Design, I did my best to provide some workable styling (without custom overrides), but couldn't do more without tearing my hair out - if anyone wants to tackle the rest, be my guest:

Screenshot from 2025-07-23 06-50-32

For the sake of comparison, this is how the enabled button looks like now:

Screenshot from 2025-07-23 06-50-44

For users who can edit available category post types, we now redirect directly to the page where they can manage them:

Screenshot from 2025-07-23 07-02-58

The "0 types" case is also fixed just in case (even through it should now not be possible to land there by accident):

Screenshot from 2025-07-23 07-02-41

Here's how the flow will look like:

ssr-2025-07-23_07.23.00.mp4

@Oaphi Oaphi added the status: blocked This is being worked on but is blocked by something else label Jul 23, 2025
@Oaphi Oaphi requested review from a team, ArtOfCode- and cellio July 23, 2025 04:25
Base automatically changed from 0valt/1509/search_tests to develop July 23, 2025 10:51
@ArtOfCode- ArtOfCode- merged commit 9d3670c into develop Jul 23, 2025
@ArtOfCode- ArtOfCode- deleted the 0valt/categories branch July 23, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: blocked This is being worked on but is blocked by something else

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants