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

fix in log migration logic #12133

Closed
wants to merge 3 commits into from
Closed

fix in log migration logic #12133

wants to merge 3 commits into from

Conversation

bumstik
Copy link

@bumstik bumstik commented Aug 10, 2016

Q A
Is bugfix? yes
New feature? no
Breaks BC? yes/no
Tests pass? yes/no
Fixed issues #11573

@samdark samdark added type:bug Bug status:to be verified Needs to be reproduced and validated. labels Aug 10, 2016
@samdark samdark self-assigned this Aug 10, 2016
@samdark samdark added this to the 2.0.11 milestone Aug 10, 2016
@bumstik
Copy link
Author

bumstik commented Sep 23, 2016

День добрый.
От меня что еще нужно что бы попало в продакшн?

@SilverFire
Copy link
Member

Пока нет. PR запланирован на следующий milestone

@samdark samdark added the pr:request for unit tests Unit tests are needed. label Dec 1, 2016
@yii-bot
Copy link

yii-bot commented Dec 1, 2016

Thank you for putting effort in the improvement of the Yii framework.
We have reviewed your pull request.

In order for the framework and your solution to remain stable in the future, we have a unit test requirement in place. Therefore we can only accept your pull request if it is covered by unit tests.

Could you add these please?

Thanks!

P.S. If you have any questions about the creation of unit tests? Don't hesitate to ask for support. More information about unit tests

This is an automated comment, triggered by adding the label pr:request for unit tests.

@SilverFire
Copy link
Member

SilverFire commented Dec 2, 2016

@bumstik вы желаете продолжить работу над этим пул-реквестом?

@samdark samdark removed the pr:request for unit tests Unit tests are needed. label Dec 2, 2016
@samdark
Copy link
Member

samdark commented Dec 2, 2016

Про тесты вышла ошибка. Тут они не нужны.

@vitalyzhakov
Copy link

Привет!
Чего не хватает для принятия в master?

@samdark
Copy link
Member

samdark commented Dec 4, 2016

CHANGELOG.

@samdark samdark removed the status:to be verified Needs to be reproduced and validated. label Dec 4, 2016
@bumstik
Copy link
Author

bumstik commented Dec 13, 2016

Получилось три коммита, сойдет?

@resurtm
Copy link
Contributor

resurtm commented Dec 13, 2016

@bumstik Нормально. При слиянии можно все три коммита в один объединить.

@samdark samdark closed this Jan 15, 2017
samdark pushed a commit that referenced this pull request Jan 15, 2017
…m141106_185632_log_init` that would create a log table correctly
@samdark
Copy link
Member

samdark commented Jan 15, 2017

Merged in 3ea6714 with some adjustments. Thank you.

lav45 added a commit to lav45/yii2 that referenced this pull request Jan 19, 2017
* master: (54 commits)
  Suggest latest PHP 7 usage in the guide [skip ci]
  Update .travis.yml
  Update .travis.yml
  travis_retry for mysql
  Fixes yiisoft#13401: Fixed lack of escaping of request dump at exception screens
  Added support for field `yii\console\controllers\BaseMigrateController::$migrationNamespaces` setup from CLI
  Adjusted changelog
  docs/guide-ja some files updated [ci skip] (yiisoft#13396)
  Revert "Fixed some comment issues (yiisoft#13392) [skip ci]"
  Fixes yiisoft#11464: Foreign key name is not provided by schema
  Removed unused method
  Exposed number normalization as StringHelper::normalizeNumber()
  Fixes yiisoft#13287: Fixed translating "and" separator in `UniqueValidator` error message
  Fixes yiisoft#12133: Fixed `getDbTargets()` function in `yii\log\migrations\m141106_185632_log_init` that would create a log table correctly
  Fixed some comment issues (yiisoft#13392) [skip ci]
  Fixed typos in docs/internals/core-code-style (yiisoft#13391) [skip ci]
  fixed some links in doc (yiisoft#13390)
  Used is_subclass_of() as suggested by @SamMousa
  Fixes yiisoft#13387: Extracted "Custom validator for multiple attributes" into community cookbook
  Updated travis config to build only on master branches
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants