Skip to content

Commit

Permalink
fix(reports): client-side validation Account Report
Browse files Browse the repository at this point in the history
This commit fixes the account report validation triggers for the account
report.  It also excludes title accounts from the account report.
  • Loading branch information
jniles committed Aug 4, 2017
1 parent 04f4561 commit 6bcf32b
Showing 1 changed file with 18 additions and 24 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,34 +23,28 @@ <h3 class="text-capitalize" translate>REPORT.REPORT_ACCOUNTS.TITLE</h3>
<div class="panel-body">
<form name="ConfigForm" bh-submit="ReportConfigCtrl.preview(ConfigForm)" novalidate>



<!-- account selection -->

<bh-account-select
id="account-id"
account-id="ReportConfigCtrl.account"
label="FORM.SELECT.ACCOUNT"
name="account"
on-select-callback="ReportConfigCtrl.selectAccount(account)"
required='true'
validation-trigger = 'ConfigForm.$submitted'
>
</bh-account-select>

<bh-account-select
id="account-id"
account-id="ReportConfigCtrl.account"
label="FORM.SELECT.ACCOUNT"
name="account"
on-select-callback="ReportConfigCtrl.selectAccount(account)"
exclude-title-accounts="true"
required="true"
validation-trigger="ConfigForm.$submitted">
</bh-account-select>

<!-- included Date interval -->

<!-- Date interval -->
<!-- @TODO this should use a component that callsback with well defined dates -->
<!-- @TODO this should use a component that callback with well defined dates -->
<bh-date-interval
date-from="ReportConfigCtrl.reportDetails.dateFrom"
date-to="ReportConfigCtrl.reportDetails.dateTo"
required
validation-trigger = 'ConfigForm.$submitted'
>
</bh-date-interval>

date-from="ReportConfigCtrl.reportDetails.dateFrom"
date-to="ReportConfigCtrl.reportDetails.dateTo"
required="true"
validation-trigger="ConfigForm.$submitted">
</bh-date-interval>

<bh-loading-button loading-state="ConfigForm.$loading">
<span translate>REPORT.UTIL.PREVIEW</span>
</bh-loading-button>
Expand Down

0 comments on commit 6bcf32b

Please sign in to comment.