Skip to content

add flatrates to billable controlling (64286)#293

Draft
svenwey wants to merge 51 commits intomasterfrom
feature/64286-add-flatrates-to-billable-controlling
Draft

add flatrates to billable controlling (64286)#293
svenwey wants to merge 51 commits intomasterfrom
feature/64286-add-flatrates-to-billable-controlling

Conversation

@svenwey
Copy link
Copy Markdown
Collaborator

@svenwey svenwey commented Jun 20, 2025

depends on #262 and #289 .
Adds flatrates to billable controlling tab for a better overview of billed / unbilled flatrates

svenwey added 30 commits April 29, 2025 08:59
…rate_invoice itself has unwanted side effects upon editing (old value used by form)
@svenwey svenwey requested a review from Kagemaru June 20, 2025 07:26
@svenwey svenwey self-assigned this Jun 20, 2025
= fields.input_field(:amount, required: true)
.col-md-1.form-action
= fields.remove_nested_fields_link(picon('delete', title: 'Pauschale entfernen'))
.row{style: "margin-bottom: 1em"}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[HAML-Lint] reported by reviewdog 🐶
[W] InlineStyles: Do not use inline style attributes

= fields.label :periodicity, "Periodizität *", class: 'col-md-2 control-label'
.col-md-10
- 0.upto(11) do |i|
.field.col-md-3{style: "margin-bottom: 0.5em"}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[HAML-Lint] reported by reviewdog 🐶
[W] InlineStyles: Do not use inline style attributes

- if (max_quantity == 0)
- display = 'none'

.invoice-flatrate{ style: "display: #{display};" }
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[HAML-Lint] reported by reviewdog 🐶
[W] InlineStyles: Do not use inline style attributes

@Kagemaru Kagemaru force-pushed the master branch 4 times, most recently from ef26473 to 89a062c Compare November 18, 2025 23:06
@Kagemaru Kagemaru force-pushed the master branch 4 times, most recently from d08e2d4 to 0750af0 Compare April 9, 2026 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant