NPM Resource Changed PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
30 Aug 2023

Pull Request for Issue #41527 .

Summary of Changes

TinyMCE v6 has strict types for the options, now it needs a setter and a getter

Testing Instructions

Try to upload an image

Actual result BEFORE applying this Pull Request

Broken

Expected result AFTER applying this Pull Request

Not broken

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

@brianteeman if you find another such error please tag me

avatar dgrammatiko dgrammatiko - open - 30 Aug 2023
avatar dgrammatiko dgrammatiko - change - 30 Aug 2023
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Aug 2023
Category JavaScript Repository NPM Change
avatar brianteeman brianteeman - test_item - 30 Aug 2023 - Tested successfully
avatar brianteeman
brianteeman - comment - 30 Aug 2023

I have tested this item ✅ successfully on 7228be4


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41528.

avatar Quy Quy - test_item - 30 Aug 2023 - Tested successfully
avatar Quy
Quy - comment - 30 Aug 2023

I have tested this item ✅ successfully on 7228be4


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41528.

avatar Quy Quy - change - 30 Aug 2023
Status Pending Ready to Commit
Labels Added: NPM Resource Changed PR-5.0-dev
avatar Quy
Quy - comment - 30 Aug 2023

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41528.

avatar dgrammatiko dgrammatiko - change - 31 Aug 2023
Labels Added: ?
avatar dgrammatiko
dgrammatiko - comment - 31 Aug 2023

@brianteeman @Quy I made a mistake, the options should be registered at the plugin level not the main instance of tinyMCE. Also the plugin was transformed to ESM so it can be linted.

If you don't mind please rede your tests (sorry)

avatar dgrammatiko dgrammatiko - change - 31 Aug 2023
Labels Removed: ?
avatar laoneo laoneo - change - 31 Aug 2023
Status Ready to Commit Pending
avatar laoneo
laoneo - comment - 31 Aug 2023

Back to pending


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41528.

avatar Quy Quy - test_item - 31 Aug 2023 - Tested successfully
avatar Quy
Quy - comment - 31 Aug 2023

I have tested this item ✅ successfully on 2464ad4


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41528.

avatar HLeithner HLeithner - close - 2 Sep 2023
avatar HLeithner HLeithner - merge - 2 Sep 2023
avatar HLeithner HLeithner - change - 2 Sep 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-09-02 16:08:44
Closed_By HLeithner
avatar HLeithner
HLeithner - comment - 2 Sep 2023

thanks

Add a Comment

Login with GitHub to post a comment