Feeling Lucky
?
Maintainers Checked
PR-4.3-dev
[#39168] - [4.3] Migrate quickicon downloadkey plugin to services
- Fixed in Code Base
- 9 Nov 2022
- Medium
- Build: 4.3-dev
- # 39168
- Diff
- heelc29:plugins/quickicon/downloadkey
User tests: Successful: Unsuccessful:
Summary of Changes
Migrate the quickicon downloadkey plugin to service providers and use SubscriberInterface.
Testing Instructions
- Install an extension with providing a download key
- Check if quickicons are still working
Actual result BEFORE applying this Pull Request
Downloadkey quickicon is working
Expected result AFTER applying this Pull Request
Downloadkey quickicon is working (and the other quickicons are still displayed)
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
laoneo
- comment
- 8 Nov 2022
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39168.
| Labels |
Added:
PR-4.3-dev
|
||
heelc29
- comment
- 8 Nov 2022
Guess this here should be lower case as well
Done
Abernyte-Git
- comment
- 8 Nov 2022
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39168.
| Status | Pending | ⇒ | Ready to Commit |
alikon
- comment
- 8 Nov 2022
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39168.
| Labels |
Added:
?
Maintainers Checked
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-11-09 17:08:06 |
| Closed_By | ⇒ | obuisard |
heelc29
- comment
- 9 Nov 2022
Thanks
Guess this here should be lower case as well https://github.com/joomla/joomla-cms/pull/39168/files#diff-6886871adeab8130a00db34ba210255b45c247430a320dc497c4ffd88c06ee13R5