Skip to content

winter 1.2.8 support#23

Merged
LukeTowers merged 17 commits into
mainfrom
wip/winter-1.2.8-support
Jul 16, 2025
Merged

winter 1.2.8 support#23
LukeTowers merged 17 commits into
mainfrom
wip/winter-1.2.8-support

Conversation

@mjauvin

@mjauvin mjauvin commented May 5, 2025

Copy link
Copy Markdown
Member

Depends on wintercms/winter#1312

Update Winter.Redirect to use formcontroller behavior's default views.

Comment thread controllers/Redirects.php
Comment on lines +111 to +117
public function create()
{
$this->formLayout = 'sidebar';

parent::create();
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is this required if we're already setting the formLayout to sidebar by default?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It was meant to be 'standard' in create context... I commited a fix.

Comment thread controllers/redirects/_form_create_toolbar.php

@LukeTowers LukeTowers left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved pending merge of wintercms/winter#1312 and release of Winter v1.2.8

@LukeTowers LukeTowers merged commit 4ea9bbb into main Jul 16, 2025
4 checks passed
@LukeTowers LukeTowers deleted the wip/winter-1.2.8-support branch July 16, 2025 23:01
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.

2 participants