I like this suggestion, too, as that's also the result you get from a tag that doesn't exist at all, so it does effectively delete/delist the tag.
I think in addition, it would be nice for tags with 0 posts to be undiscoverable except by admins using the tag manager, so even if there aren't a lot of posts on a Forem the 0 posts tags don't appear in Top Tags or similar lists.
Since tags get created whenever they are included in a post, the tag exists even with no published posts.
It has been determined that the expected behavior should be a 404 response. If the tag is supported than the expectation is that the admin wants it to show up, even with no posts. (Likely a no posts yet scenario.)
I also really liked that suggestion above to only show supported tags in the /tags page. Would be nice to be able to focus on the important tags but maybe also have a search feature to quickly navigate to any other tag you might be interested in.
For further actions, you may consider blocking this person and/or reporting abuse
I like this suggestion, too, as that's also the result you get from a tag that doesn't exist at all, so it does effectively delete/delist the tag.
I think in addition, it would be nice for tags with 0 posts to be undiscoverable except by admins using the tag manager, so even if there aren't a lot of posts on a Forem the 0 posts tags don't appear in Top Tags or similar lists.
Merged the 404 for 0 posts/unsupported tags. Will be live in Forem Cloud shortly.
Return 404 if empty and unsupported tag #14864
What type of PR is this? (check all applicable)
Description
Since tags get created whenever they are included in a post, the tag exists even with no published posts.
It has been determined that the expected behavior should be a
404
response. If the tag is supported than the expectation is that the admin wants it to show up, even with no posts. (Likely a no posts yet scenario.)Related Tickets & Documents
Closes github.com/forem/rfcs/issues/282
Originally came out of this discussion: forem.dev/ildi/is-it-possible-to-d...
Added/updated tests?
[Forem core team only] How will this change be communicated?
This should mostly just fill expected behavior.
Thanks for the update Ben!
I also really liked that suggestion above to only show supported tags in the
/tags
page. Would be nice to be able to focus on the important tags but maybe also have a search feature to quickly navigate to any other tag you might be interested in.