StrafesNET/maps-service
5
Fork 4
Code Issues 17 Pull Requests Packages Projects Releases Wiki Activity

Check that mapfixes do not change DisplayName, Creator, GameID #324

Merged
Quaternions merged 3 commits from mapfix-checks into staging 2026-06-11 18:23:48 +00:00
Conversation 1 Commits 3 Files Changed 10 +257 -93
Quaternions commented 2026-01-07 19:44:29 +00:00
Owner
Copy Link

Good lord this code is horrible. The nested check() functions constantly swapping argument order back and forth bruh

- Depends on #323 - Closes #189 Good lord this code is horrible. The nested check() functions constantly swapping argument order back and forth bruh
Quaternions added the validator label 2026-01-07 19:44:29 +00:00
Quaternions added a new dependency 2026-01-07 19:44:37 +00:00
#323 Limit DisplayName and Creator to 50 characters
Quaternions changed target branch from 50-chars to staging 2026-06-11 18:11:56 +00:00
Quaternions added 3 commits 2026-06-11 18:14:12 +00:00
backend: plumb target info into checks 2542982ff8
validation: generalize StringEquality to EqualityCheck b6d51b30b1
validation: Check that mapfixes do not change DisplayName, Creator, GameID
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details
d7f699e126
Quaternions force-pushed mapfix-checks from 474655f4a3 to d7f699e126 2026-06-11 18:14:12 +00:00 Compare
Quaternions commented 2026-06-11 18:23:02 +00:00
Author
Owner
Copy Link

Surely I refactor it at some point

Surely I refactor it at some point
Quaternions merged commit 29e391bae6 into staging 2026-06-11 18:23:48 +00:00
Quaternions deleted branch mapfix-checks 2026-06-11 18:23:48 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
backend
Submissions api related
 bug
Something is not working
 combobulator
duplicate
This issue or pull request already exists
 enhancement
New feature
 frontend
Website related
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 validator
Map validation related
 wontfix
This won't be fixed
No Label validator
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aidan9382 CI dowoge fiveman1 ic3w0lf22 interpreterK itzaname Quaternions
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Depends on
#323 Limit DisplayName and Creator to 50 characters
StrafesNET/maps-service
Reference: StrafesNET/maps-service#324
Delete Branch "mapfix-checks"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?