Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Add rectification period dgfOtherAssets #112

Closed
wants to merge 9 commits into from
Closed

WIP: Add rectification period dgfOtherAssets #112

wants to merge 9 commits into from

Conversation

bdmbdsm
Copy link
Contributor

@bdmbdsm bdmbdsm commented Aug 8, 2018

Bohdan Dmytriv added 2 commits August 1, 2018 13:43
It's probably outdated, and also takes a lot of lines.
@bdmbdsm bdmbdsm self-assigned this Aug 8, 2018
@coveralls
Copy link

coveralls commented Aug 8, 2018

Coverage Status

Coverage increased (+0.4%) to 81.995% when pulling 3e2e7f5 on bdmbdsm:add-rectification-period into 9cb5fb4 on openprocurement:master.

@bdmbdsm bdmbdsm requested a review from leits August 8, 2018 10:59
@bdmbdsm bdmbdsm changed the title Add rectification period Add rectification period dgfOtherAssets Aug 8, 2018
@@ -10,3 +10,5 @@

DEFAULT_PROCUREMENT_METHOD_TYPE_OTHER = "DGFOtherAssets"
DEFAULT_PROCUREMENT_METHOD_TYPE_FINANCIAL = "DGFFinancialAssets"

RECTIFICATION_PERIOD_LENGTH_DAYS = 2
Copy link
Member

Choose a reason for hiding this comment

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

Тут одразу повинна бути timedelta i змінна тоді RECTIFICATION_PERIOD_DURATION

now = get_now()
if self.status == 'active.tendering':
if (
now >= self.rectificationPeriod.startDate
Copy link
Member

Choose a reason for hiding this comment

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

Чому б не використати тут впроваджене тобою in?

Copy link
Member

Choose a reason for hiding this comment

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

Це якщо є цей період. У старих процедур його не буде

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Точно! Я на цьому місці його і придумав, а тепер забув.

@bdmbdsm bdmbdsm changed the title Add rectification period dgfOtherAssets WIP: Add rectification period dgfOtherAssets Aug 15, 2018
@bdmbdsm bdmbdsm changed the title WIP: Add rectification period dgfOtherAssets Add rectification period dgfOtherAssets Aug 15, 2018
@bdmbdsm bdmbdsm changed the title Add rectification period dgfOtherAssets WIP: Add rectification period dgfOtherAssets Aug 15, 2018
@bdmbdsm bdmbdsm closed this Aug 16, 2018
@bdmbdsm bdmbdsm deleted the add-rectification-period branch August 16, 2018 14:04
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.

None yet

3 participants