C2025-010_Ability-to-Toggle-Replace-Words-2 #501

Merged
jjenko merged 1 commits from C2019-025_Ability-to-Toggle-Replace-Words-2 into Development 2025-01-29 15:04:10 -05:00
Owner

C2025-010 - Made adjustments in the step settings for the drop-down values for "Show Replace Words".

C2025-010 - Made adjustments in the step settings for the drop-down values for "Show Replace Words".
plarsen self-assigned this 2025-01-29 15:00:38 -05:00
jjenko was assigned by plarsen 2025-01-29 15:00:38 -05:00
mvickers was assigned by plarsen 2025-01-29 15:00:38 -05:00
plarsen added 1 commit 2025-01-29 15:00:39 -05:00
plarsen requested review from jjenko 2025-01-29 15:00:50 -05:00
mschill reviewed 2025-01-29 15:03:42 -05:00
mschill left a comment
Owner

One question - these are just numbers in the DisplayText.cs ---- any chance we could either use an enum type or add comments saying what each number means / corresponds to?

One question - these are just numbers in the DisplayText.cs ---- any chance we could either use an enum type or add comments saying what each number means / corresponds to?
jjenko approved these changes 2025-01-29 15:03:56 -05:00
jjenko left a comment
Owner

changes look good.

changes look good.
jjenko merged commit fe268b6122 into Development 2025-01-29 15:04:10 -05:00
Owner

sorry I merged this in too fast.
I agree with Matt that use of Enum or comments should be done.

sorry I merged this in too fast. I agree with Matt that use of Enum or comments should be done.
djankowski changed title from C2019-025_Ability-to-Toggle-Replace-Words-2 to C2025-020_Ability-to-Toggle-Replace-Words-2 2025-01-29 15:09:13 -05:00
djankowski changed title from C2025-020_Ability-to-Toggle-Replace-Words-2 to C2025-010_Ability-to-Toggle-Replace-Words-2 2025-01-29 15:10:14 -05:00
Sign in to join this conversation.
No Reviewers
No Label
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Volian/SourceCode#501
No description provided.