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

Rename register component twig function to prepare #30

Conversation

alexander-schranz
Copy link
Member

@alexander-schranz alexander-schranz commented Sep 25, 2020

Fixes #29

This will rename the following functions

register_component -> prepare_component
call_service -> prepare_service

@alexander-schranz alexander-schranz force-pushed the enhancement/rename-register-component-to-prepare-component branch 2 times, most recently from b1b23af to 32e3487 Compare September 25, 2020 08:30
@alexander-schranz alexander-schranz force-pushed the enhancement/rename-register-component-to-prepare-component branch from 32e3487 to 86d90bb Compare September 25, 2020 08:45
@alexander-schranz alexander-schranz force-pushed the enhancement/rename-register-component-to-prepare-component branch from 86d90bb to d185d1a Compare September 25, 2020 08:45
@alexander-schranz alexander-schranz marked this pull request as ready for review February 25, 2021 11:00
docs/component.md Outdated Show resolved Hide resolved
docs/component.md Outdated Show resolved Hide resolved
docs/component.md Show resolved Hide resolved
docs/component.md Outdated Show resolved Hide resolved
@@ -30,7 +30,7 @@ public function setUp(): void

public function testRegisterComponent(): void
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe adjust test names?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good catch did update them 👍

tests/Unit/ComponentExtensionTest.php Outdated Show resolved Hide resolved
alexander-schranz and others added 7 commits February 25, 2021 12:23
Co-authored-by: nnatter <niklas.natter@gmail.com>
Co-authored-by: nnatter <niklas.natter@gmail.com>
Co-authored-by: nnatter <niklas.natter@gmail.com>
Co-authored-by: nnatter <niklas.natter@gmail.com>
Co-authored-by: nnatter <niklas.natter@gmail.com>
@niklasnatter niklasnatter merged commit 789a019 into sulu:master Feb 25, 2021
@alexander-schranz alexander-schranz deleted the enhancement/rename-register-component-to-prepare-component branch February 25, 2021 11:32
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.

Renaming the register_component twig function
2 participants