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

Update Prerequisites #186

Merged
merged 1 commit into from
Oct 7, 2024
Merged

Update Prerequisites #186

merged 1 commit into from
Oct 7, 2024

Conversation

linawolf
Copy link
Member

@linawolf linawolf commented Oct 6, 2024

And move them to their own page

Releases: main

Copy link
Contributor

@brotkrueml brotkrueml left a comment

Choose a reason for hiding this comment

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

I suggest to move at least some content from the introduction to the start page, especially, WHY a site package is recommended, like encapsulation and security (storing templates/css/js files in fileadmin is discouraged).

Documentation/Prerequisites/Index.rst Outdated Show resolved Hide resolved
Documentation/Prerequisites/Index.rst Show resolved Hide resolved
@linawolf
Copy link
Member Author

linawolf commented Oct 6, 2024

This is a Tutorial so it should be followed step by step by beginners. The theories are neither helpful to beginners nor will they most probably be read.

The pros and cons of using a site package can be described in TYPO3 explained for advanced developers. Here we simply explain one way of how to do things.

With the recent changes it has become possible to create a site without sitepackage by having files in the site folder. Having templates in the fileadmin is completely outdated. We do not want to confuse newcomers with this.

If you feel any of the information is valuable enough you can update it and move it to a more suitable location

Copy link
Contributor

@garvinhicking garvinhicking left a comment

Choose a reason for hiding this comment

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

My oppinion: I really think this is very worthful and belongs there. A tutorial should not just be copy&paste only, but educate people what they do and why.

These are distinct pages, if people are not interested in that, they can skip it.

If this gets removed here, it should first get added somewhere else and properly linked here.

Releases: main
@linawolf
Copy link
Member Author

linawolf commented Oct 7, 2024

@garvinhicking, I reverted deleting the Preface and Introduction, let us discuss them separately.

Do you think the rest can be merged?

Copy link
Contributor

@garvinhicking garvinhicking left a comment

Choose a reason for hiding this comment

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

Thanks, that makes me happy :)

@linawolf linawolf merged commit b511878 into main Oct 7, 2024
2 checks passed
@linawolf linawolf deleted the task/preface branch October 7, 2024 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants