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

Добавления поле Weight, для правильного расчета стоимости доставки Я.Маркет #19

Closed
wants to merge 6 commits into from

Conversation

Wormaster
Copy link

Добавления поле Weight, для правильного расчета стоимости доставки Я.Маркет

Выдержка из доки Я.Маркета

weight | Вес товара в килограммах с учетом упаковки. Для некоторых категорий установлены ограничения по минимальному или максимальному значению веса. Скачать список минимальных и максимальных значений веса. В любой категории вес можно указывать с точностью до тысячных (например, 1.001 кг; разделитель целой и дробной части — точка). Если минимальное значение указано 0, ограничений по минимальному весу нет, и можно указывать начиная с одного грамма (0.001 кг). | Необязательно

@Bukashk0zzz
Copy link
Owner

@Wormaster Исправь пожалуйста код стайл и я солью.

@codecov-io
Copy link

codecov-io commented Feb 25, 2019

Codecov Report

Merging #19 into master will decrease coverage by 0.43%.
The diff coverage is 50%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #19      +/-   ##
============================================
- Coverage       100%   99.56%   -0.44%     
- Complexity      243      245       +2     
============================================
  Files            17       17              
  Lines           676      682       +6     
============================================
+ Hits            676      679       +3     
- Misses            0        3       +3
Impacted Files Coverage Δ Complexity Δ
src/Model/Offer/AbstractOffer.php 97.97% <50%> (-2.03%) 52 <2> (+2)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 98c4549...561defd. Read the comment docs.

@Bukashk0zzz
Copy link
Owner

Нужно еще добавить тест.
Нужно просто в добавить setWeight

->setOldPrice($this->faker->numberBetween(1, 9999))

@Wormaster
Copy link
Author

Хм... а вот теперь там что-то неясное отвалилось

@Bukashk0zzz
Copy link
Owner

В итоге добавил сам это поле и сделал релиз.
https://github.com/Bukashk0zzz/YmlGenerator/releases/tag/v1.6.2

@Bukashk0zzz Bukashk0zzz closed this Mar 1, 2019
@Bukashk0zzz
Copy link
Owner

Изминения 6ae4bd9

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