Skip to content

Conversation

JanGalek
Copy link

@JanGalek JanGalek commented Jun 7, 2016

Route::$defaultFlags is deprecated, so there is substitude for it.

@@ -323,6 +323,27 @@ Pokud potřebujete do HTTPS překlopit celou aplikaci, použijte `Route::$defaul
Route::$defaultFlags = Route::SECURED;
\--

.[caution]
Route::$defaultFlags v Nette 2.4 již nepodporovaté
Copy link
Contributor

Choose a reason for hiding this comment

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

NepodporovaNé ;)

@klimesf klimesf merged commit e8f6047 into nette:doc-2.4 Jun 8, 2016
@klimesf
Copy link
Contributor

klimesf commented Jun 8, 2016

Thank you! 👍

@JanGalek
Copy link
Author

JanGalek commented Jun 8, 2016

I too 👍

.[caution]
Route::$defaultFlags is deprecated since Nette 2.4

Substitude for `Route::$defaultFlags`
Copy link
Member

@dg dg Jun 8, 2016

Choose a reason for hiding this comment

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

Substitute (or replacement)

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh darn, I missed this one. I will fix it.

Copy link
Member

Choose a reason for hiding this comment

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

I already fixed it

@dg
Copy link
Member

dg commented Jun 8, 2016

Díky @JanGalek za sepsání, ale není to úplně přesné, zatím to revertnu, ok?

@klimesf
Copy link
Contributor

klimesf commented Jun 8, 2016

Omlouvám se, měl jsem počkat na další feedback :(. Chybu beru na sebe.

@dg
Copy link
Member

dg commented Jun 8, 2016

V poho :-)

@JanGalek
Copy link
Author

JanGalek commented Jun 9, 2016

@dg ok, omlouvám se

@dg
Copy link
Member

dg commented Jun 9, 2016

Mám teď šílené fofry, ale dáme ten text dohromady.

@JanGalek
Copy link
Author

JanGalek commented Jun 9, 2016

Poslal jsem na toto nový PR, poupravil jsem texty, #452

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants