Skip to content
This repository has been archived by the owner on Oct 16, 2024. It is now read-only.

Allow for UsageTypes without DailyUsages for validation #615

Merged
merged 2 commits into from
May 22, 2017

Conversation

mkurek
Copy link
Contributor

@mkurek mkurek commented May 22, 2017

No description provided.

@mkurek mkurek requested a review from xor-xor May 22, 2017 06:08
@coveralls
Copy link

coveralls commented May 22, 2017

Coverage Status

Coverage increased (+0.008%) to 84.969% when pulling 950bf61 on mkurek:ut-no-daily-usage into 626f628 on allegro:develop.

migrations.AddField(
model_name='usagetype',
name='allow_no_daily_usage',
field=models.BooleanField(default=False, help_text='Skip validation of this UsageType for existing DailyUsages beforerecalculating costs'),
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor nitpick: s/beforecalculating/before calculating/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@coveralls
Copy link

coveralls commented May 22, 2017

Coverage Status

Coverage increased (+0.008%) to 84.969% when pulling 06d5e62 on mkurek:ut-no-daily-usage into 626f628 on allegro:develop.

@mkurek mkurek merged commit 5982b5c into allegro:develop May 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants