Skip to content

feature Awarding X plugin stage 1 *draft*#225

Open
antonkorobko wants to merge 3 commits intoopenprocurement:masterfrom
antonkorobko:feature_awarding_x_plugin
Open

feature Awarding X plugin stage 1 *draft*#225
antonkorobko wants to merge 3 commits intoopenprocurement:masterfrom
antonkorobko:feature_awarding_x_plugin

Conversation

@antonkorobko
Copy link
Copy Markdown

@antonkorobko antonkorobko commented Jan 30, 2019

This change is Reviewable

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 812

  • 85 of 213 (39.91%) changed or added relevant lines in 7 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.2%) to 35.158%

Changes Missing Coverage Covered Lines Changed/Added Lines %
openprocurement/auctions/core/plugins/awarding/vX/models.py 33 59 55.93%
openprocurement/auctions/core/plugins/awarding/vX/utils.py 11 58 18.97%
openprocurement/auctions/core/plugins/awarding/vX/adapters.py 27 82 32.93%
Totals Coverage Status
Change from base Build 810: 0.2%
Covered Lines: 1779
Relevant Lines: 5060

💛 - Coveralls

1 similar comment
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 812

  • 85 of 213 (39.91%) changed or added relevant lines in 7 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.2%) to 35.158%

Changes Missing Coverage Covered Lines Changed/Added Lines %
openprocurement/auctions/core/plugins/awarding/vX/models.py 33 59 55.93%
openprocurement/auctions/core/plugins/awarding/vX/utils.py 11 58 18.97%
openprocurement/auctions/core/plugins/awarding/vX/adapters.py 27 82 32.93%
Totals Coverage Status
Change from base Build 810: 0.2%
Covered Lines: 1779
Relevant Lines: 5060

💛 - Coveralls



@implementer(IAwardVX)
class Award(BaseAward):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

З того, що я розумію то цей awarding відрізняється від awarding_3_1 довжиною періодів. Але ці періоди конфігуруються в кінцевих пакунках, як зроблено в rubble. Чи є якась причина робити копію awarding_3_1 чи тут є зміни, які не дають змогу застосувати awarding_3_1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

І мають бути міграції, щоб перевести існуючі awards 2_1 до 3_1

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.

3 participants