fix: type_id -> type_instance_id dans createResolutionsUIhandler #92

ariasat · 2026-06-22T15:35:28-04:00

ariasat commented

2026-06-22 15:35:28 -04:00

closes #81

ariasat added 1 commit

2026-06-22 15:35:28 -04:00

fix: type_id -> type_instance_id dans createResolutionsUIhandler

/ test-build (pull_request) Successful in 28s

Details

b712d40877

ariasat merged commit 4a7cbb9047 into main

2026-06-22 15:38:32 -04:00

ariasat deleted branch type_instance_fix

2026-06-22 15:38:32 -04:00

ariasat referenced this pull request from a commit

2026-06-22 15:38:33 -04:00

Merge pull request 'fix: type_id -> type_instance_id dans createResolutionsUIhandler' (#92) from type_instance_fix into main

dnova reviewed

2026-06-22 15:39:00 -04:00

handler_ui.go

					
				@ -522,3 +522,3 @@

						}

						typeID := getUUIDFromForm(r, "type_id")

						typeID := getUUIDFromForm(r, "type_instance_id")

ISSUE: Variable name "typeID" doesn't match form parameter "type_instance_id"
Consider renaming to "typeInstanceID" for consistency, or ensure downstream code
understands this now refers to an instance, not a type definition.
Also clarify the semantic difference between type_instance_id vs instance_id.

ISSUE: Variable name "typeID" doesn't match form parameter "type_instance_id" Consider renaming to "typeInstanceID" for consistency, or ensure downstream code understands this now refers to an instance, not a type definition. Also clarify the semantic difference between type_instance_id vs instance_id.

has been resolved in the future

🚀 1

Sign in to join this conversation.

No reviewers

No labels

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

agecem/resolutions!92

Rows
Columns