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

deleting a category with assigned articles does not return an error message. #1728

Closed
phiw13 opened this issue Aug 18, 2021 · 6 comments
Closed
Milestone

Comments

@phiw13
Copy link

phiw13 commented Aug 18, 2021

See this forum post: https://forum.textpattern.com/viewtopic.php?pid=331405#p331405 for details

apparently no string exist for an eventual error message

@bloatware
Copy link
Member

apparently no string exist for an eventual error message

and that's the main obstacle.

@phiw13
Copy link
Author

phiw13 commented Aug 19, 2021

Sounds like an easy fix then for someone with access to insert the string, then.

4.8.8 polish stuff?

@petecooper
Copy link
Member

@Bloke @bloatware - viable candidate for 4.8.8?

@Bloke
Copy link
Member

Bloke commented Sep 1, 2021

Yep.

@Bloke Bloke added this to the v4.8.8 milestone Sep 1, 2021
bloatware added a commit that referenced this issue Sep 9, 2021
@phiw13
Copy link
Author

phiw13 commented Sep 13, 2021

This seems to work after your patch (5417c88). Attempting to delete a category with child categories or with articles assigned now returns a notification message flagging the failure.

One might quibble about the type of message. Currently I see a warning (amber). Trying to delete a section with articles on the other hand returns an error(red).

@bloatware
Copy link
Member

Fair enough, done, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants