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

Closed
dbuhonov wants to merge 10 commits into from
Closed

Add widgets configuration file (#436) #452

Show file tree
Hide file tree
Changes from 9 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 #436: Add widgets configuration file (@dbuhonov)
- 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' => [

@vjik vjik Mar 11, 2026

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.

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

@dbuhonov dbuhonov Mar 11, 2026

Copy link
Copy Markdown
Contributor Author

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?

@dbuhonov dbuhonov Mar 17, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

up pls

@vjik vjik Mar 17, 2026

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.

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?

Oh, sorry, I missed it 😞. In this case this issue is not actual.

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

@vjik vjik Mar 11, 2026

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.

Do we need widgets for the console?

@dbuhonov dbuhonov Mar 11, 2026

Copy link
Copy Markdown
Contributor Author

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