Skip to content

Comments

feat: Remove order fields from all models that have it#631

Merged
jforsman merged 2 commits intomasterfrom
remove_order_field
Sep 1, 2025
Merged

feat: Remove order fields from all models that have it#631
jforsman merged 2 commits intomasterfrom
remove_order_field

Conversation

@jforsman
Copy link
Contributor

@jforsman jforsman commented Aug 6, 2025

Height fields is used instead, order is just duplicate information so it can be removed.

Refs: LIIK-416

@jforsman jforsman force-pushed the remove_order_field branch from c530f77 to b793eb9 Compare August 6, 2025 12:28
Copy link
Contributor

@shundread shundread left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Height fields is used instead, order is just duplicate information so it can be removed.

Refs: LIIK-416
* Affect all models that uses BaseAdditionalSignInline as an inline

Refs: LIIK-416
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 1, 2025

@jforsman jforsman merged commit 766a2a6 into master Sep 1, 2025
3 checks passed
@jforsman jforsman deleted the remove_order_field branch September 1, 2025 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants