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

[Taxon] Edit taxon slug #6657

Merged
merged 7 commits into from
Nov 9, 2016
Merged

Conversation

Zales0123
Copy link
Member

Q A
Bug fix? no
New feature? yes
BC breaks? yes
Related tickets based on #6597
License MIT

@pjedrzejewski pjedrzejewski added BC Break PRs introducing BC breaks (do not even try to merge). New Feature labels Nov 8, 2016
@Zales0123 Zales0123 changed the title [WIP][Taxon] Edit taxon slug [Taxon] Edit taxon slug Nov 8, 2016
@Zales0123 Zales0123 force-pushed the edit-taxons-slug branch 3 times, most recently from 5d51246 to 699401c Compare November 8, 2016 21:41
@pjedrzejewski pjedrzejewski merged commit 6a6aa6c into Sylius:master Nov 9, 2016
@pjedrzejewski
Copy link
Member

Thank you Mateusz! :)

private function waitForSlugGenerationIfNecessary()
{
if ($this->getDriver() instanceof Selenium2Driver) {
$this->getDocument()->waitFor(1000, function () {
Copy link
Contributor

Choose a reason for hiding this comment

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

Waiting for 1000 seconds seems like quite too much.

Copy link
Member Author

Choose a reason for hiding this comment

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

Free commit 😄

}

$value = $slugElement->getValue();
$this->getDocument()->waitFor(1000, function () use ($slugElement, $value) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BC Break PRs introducing BC breaks (do not even try to merge).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants