Tours: drop debug log; close addressable-events audit (1.2.32)
Removed the leftover console.log of the full tour event payload in CreateTourPage — the last of the "Publishing … event" debug logs (spot, shop, tour all now clean). Audit findings for Tours (no code change beyond the log): - Tours (kind 30823) are create-only with a timestamped identifier, so there is NO edit path and therefore no edit-clobber risk to fix. (There's also no EditTourPage / /edit-tour route at all — a genuine UX gap where an organizer can't fix a typo/price/meeting-point without deleting and recreating, but building an editor for the large multi-category tour form is a feature, not an audit fix, and rushing it would risk the exact tag-clobber bugs just fixed elsewhere. Leaving it flagged rather than half-doing it.) - Owner/admin management is already correct: TourDetailPage renders DeleteEntityDialog, which internally gates to owner-or-admin. This wraps the addressable-events pass (spots, events, shops, tours, Lab profiles, crews, marketplace): every EDIT path either preserves unmanaged tags or was confirmed safe, and every entity has a working owner management path except tour-editing, which is now explicitly flagged as future work. Co-authored-by: shakespeare.diy <assistant@shakespeare.diy>
This commit is contained in:
parent
7554a39379
commit
a9f9931430
2 changed files with 1 additions and 7 deletions
|
|
@ -1,7 +1,7 @@
|
|||
{
|
||||
"name": "skate-spots",
|
||||
"private": true,
|
||||
"version": "1.2.31",
|
||||
"version": "1.2.32",
|
||||
"type": "module",
|
||||
"homepage": "https://skate-spots.com",
|
||||
"scripts": {
|
||||
|
|
|
|||
|
|
@ -712,12 +712,6 @@ export default function CreateTourPage() {
|
|||
tags.push(['featured', String(featuredUntilSeconds)]);
|
||||
}
|
||||
|
||||
console.log('Publishing tour event:', {
|
||||
kind: SKATE_TOUR_KIND,
|
||||
tags,
|
||||
content: description.trim(),
|
||||
});
|
||||
|
||||
await createEvent({
|
||||
kind: SKATE_TOUR_KIND,
|
||||
content: description.trim(),
|
||||
|
|
|
|||
Loading…
Reference in a new issue