testserver protect_quest allows empty area #4482

Closed
opened 2023-05-13 14:27:12 +00:00 by AliasAlreadyTaken · 3 comments

Please make it mandatory to have an area name.

Please make it mandatory to have an area name.
AliasAlreadyTaken added the
1. kind/bug
label 2023-05-13 14:27:18 +00:00
AliasAlreadyTaken added this to the 1.1.119 milestone 2023-05-13 14:32:39 +00:00
Member

it defaults the area name to "Quest", doesn't it? which is what most of the quest areas are named?

it defaults the area name to "Quest", doesn't it? which is what most of the quest areas are named?
Member

oh, looks like i munged the check for an empty name, fixed in f974ce9630. i can also change the behavior to require an name explicitly, but it should now work as i intended.

oh, looks like i munged the check for an empty name, fixed in https://gitea.your-land.de/your-land/yl_areas_addon/commit/f974ce96306c3a9330e76f0e520701ba353874fb. i can also change the behavior to require an name explicitly, but it should now work as i intended.
flux added the
4. step/ready to QA test
label 2023-05-13 17:05:29 +00:00
AliasAlreadyTaken added the
ugh/QA OK
label 2023-05-14 15:58:15 +00:00
flux added
5. result/fixed
and removed
4. step/ready to QA test
labels 2023-05-18 23:35:59 +00:00
Member

this is live

this is live
flux closed this issue 2023-05-18 23:36:04 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: your-land/bugtracker#4482
No description provided.