Skip to content

Conversation

@Hawkeye1909
Copy link
Contributor

@Hawkeye1909 Hawkeye1909 commented Apr 6, 2023

Pull Request

Related Issues

Prerequisites

  • Changes have been tested on TYPO3 v10.4 LTS
  • Changes have been tested on TYPO3 v11.5 LTS

Description

As EXT:felogin is available as Extbase plugin using Fluid templates since Typo3 v10.2 (https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/10.2/Feature-88102-FrontendLoginViaFluidAndExtbase.html) EXT:bootstrap_package should provide templates for it. The templates are based on the old one for the PiBase plugin.

Steps to Validate

  1. Activate feature toggle "Felogin: extbase"
  2. Add login plugin to website

Copy link
Contributor

@linawolf linawolf left a comment

Choose a reason for hiding this comment

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

These changes work great in my project where I just updated from TYPO3 v10.4 to 11.5

@benjaminkott
Copy link
Owner

We cannot take this patch in, v12 has changed templates, that also set attributes on view helpers that do not exist in v11.

Copy link
Owner

@benjaminkott benjaminkott left a comment

Choose a reason for hiding this comment

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

We cannot handle login templates, by default since we maintain compact with 2 LTS versions.

@benjaminkott
Copy link
Owner

still same remarks, merged the changes to a single patchset for easier rebase

@benjaminkott
Copy link
Owner

We keep the patch around till we drop support for v11

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

felogin not working? Obsolete config and templates for EXT:felogin

3 participants