8000 Move stopwatch docs by javiereguiluz · Pull Request #12291 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

Move stopwatch docs #12291

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

Closed
wants to merge 2 commits into from
Closed

Conversation

javiereguiluz
Copy link
Member

No description provided.

@@ -445,4 +445,5 @@
/console/request_context /routing
/form/action_method /forms
/reference/requirements /setup
/bundles/inheritance /bundles/override
/bundles/inheritance /bundles/override
/components/stopwatch https://symfony.com/components/Stopwatch
Copy link
Member

Choose a reason for hiding this comment

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

why. do we need https://symfony.com/ as this is the default?

Copy link
Member Author

Choose a reason for hiding this comment

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

Because this redirection map is:

OLD documentation page path -> NEW documentation page path

If we use /components/Stopwatch it will redirect to https://symfony.com/doc/current/components/Stopwatch.html.

We need to redirect to an entirely different URL and that's why we need absolute URLs.

@xabbuh
Copy link
Member
xabbuh commented Sep 25, 2019

@javiereguiluz Can you rebase here?

@javiereguiluz
Copy link
Member Author

I'm not going to rebase yet ... because we're waiting for the "green light" to this proposal ... so this could be discarded at the end. Let's wait a bit. Thanks.

@OskarStark
Copy link
Contributor

I am 👎🏻 against moving the docs

@wouterj wouterj mentioned this pull request Oct 4, 2020
< 8000 div class="flex-auto wb-break-word" id="ref-pullrequest-714297809" > [Stopwatch] Move component docs into framework guides #14333
Merged
@wouterj
Copy link
Member
wouterj commented Oct 4, 2020

Hi @javiereguiluz! I somehow completely missed this PR. I think the changes in this PR are great, they do not add a new guide but instead document the component where it's most useful. I've rebased this PR on 4.4 in #14333 .

@wouterj wouterj closed this Oct 4, 2020
javiereguiluz added a commit that referenced this pull request Oct 7, 2020
…aviereguiluz)

This PR was squashed before being merged into the 4.4 branch.

Discussion
----------

[Stopwatch] Move component docs into framework guides

This PR continues the great work done by @javiereguiluz in #12291 . It's mostly a rebase to 4.4.

Commits
-------

fc847c4 [Stopwatch] Move component docs into framework guides
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.

6 participants
0