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

[Typo3] assorted changes #1646

Merged
merged 4 commits into from Jun 17, 2018
Merged

[Typo3] assorted changes #1646

merged 4 commits into from Jun 17, 2018

Conversation

@josefglatz
Copy link
Contributor

@josefglatz josefglatz commented Sep 2, 2015

No description provided.

josefglatz added 4 commits Sep 2, 2015
This file is part of the configuration for TYPO3.
These temporary files should never be commited.
If you are using the common used ext:realurl without a custom extConf
the auto generated config file shouldn’t be added to scm.
@shiftkey
Copy link
Member

@shiftkey shiftkey commented Mar 6, 2016

@jousch do you have any extra details or links to help explain these changes?

@shiftkey shiftkey changed the title TYPO3 improvements [Typo3] assorted changes Jan 29, 2017
@M-arcus
Copy link

@M-arcus M-arcus commented Mar 30, 2017

I'll try to explain some of the changes:

Commit "[TYPO3] Remove AdditionalConfiguration.php"
AdditionalConfiguration.php is used for the config-changes that differ from environment to environment. Instead of editing LocalConfiguration.php you can simply switch out the AdditionalConfiguration.php while keeping the main config intact. Quite handy.

[TYPO3] Add temporary files to .gitignore
These files enable access to the install-wizard and the install-tool. They should never be comitted, because it could allow access to external users, that have no access to the file-system.

@shiftkey
Copy link
Member

@shiftkey shiftkey commented Jun 17, 2018

@M-arcus thanks!

@shiftkey shiftkey merged commit 0bce621 into github:master Jun 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants