Skip to content

Add widgets configuration file (#436) #452

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.

Sign up for GitHub

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

Open
dbuhonov wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Add widgets configuration file (#436) #452

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
eed0b70
Add widgets configuration file (#436)
dbuhonov Mar 6, 2026
511a803
Add widgets configuration file (#436)
dbuhonov Mar 6, 2026
1616635
Apply PHP CS Fixer and Rector changes (CI)
dbuhonov Mar 6, 2026
ed0916f
Merge remote-tracking branch 'origin/feature/436-widgets-config' into…
dbuhonov Mar 10, 2026
a659720
Add widgets-related configurations to the main configuration file (#436)
dbuhonov Mar 10, 2026
27d94b9
Update CHANGELOG with widgets configuration changes (#452)
dbuhonov Mar 10, 2026
26d239a
Merge branch 'master' into feature/436-widgets-config
dbuhonov Mar 10, 2026
3ea9b6e
Update CHANGELOG.md
dbuhonov Mar 11, 2026
e16920d
Merge branch 'master' into feature/436-widgets-config
vjik Mar 11, 2026
3ee122b
Add widgets configuration file (#436)
dbuhonov Mar 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

## 1.2.1 under development

- Chg #452: Add widgets configuration file (@dbuhonov)
Copy link

Copilot AI Mar 10, 2026

Choose a reason for hiding this comment

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

Changelog entry references "#452", but this PR/linked issue is #436. Please update the issue/PR number in the changelog line so it correctly points to the change being introduced here.

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

@dbuhonov dbuhonov Mar 11, 2026

Choose a reason for hiding this comment

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

Fixed

- Chg #430: Replace deprecated PHP CS Fixer rule set `@PER-CS2.0` with `@PER-CS` (@dbuhonov)
- Chg #447, #450: Allow symfony/console 8 (@samdark)
- Enh #448: Add grouping to `make` help output (@Xakki, @samdark)
Expand Down
6 changes: 6 additions & 0 deletions config/configuration.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,12 @@
'events' => [],
'events-web' => ['$events'],
'events-console' => ['$events'],
'widgets' => [],
'widgets-web' => [
Copy link
Member

@vjik vjik Mar 11, 2026

Choose a reason for hiding this comment

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

Configure Yii Widgets params. See example: https://github.com/yiisoft/demo-diary/blob/675115b0879881e117ed09dc6d77a64a93ebf38f/config/site/params.php#L17

Copy link
Contributor Author

@dbuhonov dbuhonov Mar 11, 2026

Choose a reason for hiding this comment

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

yiisoft/widget is not installed in this project, so I dont quite understand what is expected here besides adding an empty configuration file. Could you clarify what should be configured in this case?

'$widgets',
'web/widgets.php',
],
'widgets-console' => '$widgets',
Copy link
Member

@vjik vjik Mar 11, 2026

Choose a reason for hiding this comment

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

Do we need widgets for the console?

Copy link
Contributor Author

@dbuhonov dbuhonov Mar 11, 2026

Choose a reason for hiding this comment

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

No, remove

'routes' => 'common/routes.php',
'bootstrap' => 'common/bootstrap.php',
'bootstrap-web' => '$bootstrap',
Expand Down
5 changes: 5 additions & 0 deletions config/web/widgets.php
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
<?php

declare(strict_types=1);

return [];
Loading