-
Notifications
You must be signed in to change notification settings - Fork 186
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
patch(v5): pr6386 #6409
patch(v5): pr6386 #6409
Conversation
Eсли смотреть компонент `DatePicker` с переданным `monthNames` на тач устройствах, то cвойство `monthNames` встает html-аттрибутом. В режиме локальной разработки есть реакт-варнинг. Не передаем `monthNames` в компоненты ниже `NativeDatePicker`.
size-limit report 📦
|
e2e tests |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 1702ba3:
|
👀 Docs deployed
Commit 1702ba3 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## v5 #6409 +/- ##
=====================================
Coverage ? 80.73%
=====================================
Files ? 314
Lines ? 10842
Branches ? 3381
=====================================
Hits ? 8753
Misses ? 2089
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
❌ PatchНе удалось автоматически применить исправление на ветке v5.
Чтобы изменение попало в ветку v5, выполните следующие действия:
git stash # опционально
git fetch origin v5
git checkout -b patch/pr6409 origin/v5
git cherry-pick --no-commit 97acb5aa1f7c4671246ff90895e61a509d77e62c
git checkout HEAD **/__image_snapshots__/*.png
git diff --quiet HEAD || git commit --no-verify --no-edit
git push --set-upstream origin patch/pr6409
gh pr create --base v5 --title "patch: pr6409" --body "- patch #6409" |
Запутался в тэгах. Нужен был patch а не v5. Надо делать ещё и в v5-stable PR. |
…#6409) Eсли смотреть компонент `DatePicker` с переданным `monthNames` на тач устройствах, то cвойство `monthNames` встает html-аттрибутом. В режиме локальной разработки есть реакт-варнинг. Не передаем `monthNames` в компоненты ниже `NativeDatePicker`.
возможно в ветке v5 стоит убрать из workflow проставление
|
Да, я понимаю, уже просто глаз замылился. Я специально не писал fix. 🙈 Хотел сам выставить что нужно, но перепутал автоматический v5 и patch. |
Warning
см. #6410