-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
run the convert.sh script by CDK2020 to replace namespaces to be comp… #8
Conversation
src/Listener/AddClientAssets.php
Outdated
@@ -21,13 +21,13 @@ class AddClientAssets | |||
*/ | |||
public function subscribe(Dispatcher $events) | |||
{ | |||
$events->listen(ConfigureWebApp::class, [$this, 'addAssets']); | |||
$events->listen(Rendering::class, [$this, 'addAssets']); | |||
} | |||
|
|||
/** | |||
* @param ConfigureClientView $event |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update this docblock
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I'm not sure what it should be instead. New to Flarum.
I see this on the namespace changes thread:
{Event/ConfigureWebApp → Frontend/Event/Rendering}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No worries! Simply change line 28:
* @param ConfigureClientView $events
to:
* @param Rendering $events
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh duh! Think I got it, thanks.
Thanks @AakLak :D |
Nooo... this should have gone to the |
#8) * run the convert.sh script by CDK2020 to replace namespaces to be compatible for beta8 * update docbloc to match updated namespace
Oops 😳 Fixed, I think |
#8) * run the convert.sh script by CDK2020 to replace namespaces to be compatible for beta8 * update docbloc to match updated namespace
… be comp… (#8)" This reverts commit c58b60849f63666211fc47fcea3907fdb520a76b.
#8) * run the convert.sh script by CDK2020 to replace namespaces to be compatible for beta8 * update docbloc to match updated namespace
… be comp… (#8)" This reverts commit efe07bd696b66bf3dbd4a17033747a48f4b154c4.
Run this script:
https://discuss.flarum.org/d/6573-convert-your-beta-7-compatible-extension-to-work-on-beta-8