Skip to content

Comments

Feature/draggable typo fixes#29987

Merged
richard67 merged 6 commits intojoomla:4.0-devfrom
hans2103:feature/draggable-typo-fixes
Jul 20, 2020
Merged

Feature/draggable typo fixes#29987
richard67 merged 6 commits intojoomla:4.0-devfrom
hans2103:feature/draggable-typo-fixes

Conversation

@hans2103
Copy link
Contributor

@hans2103 hans2103 commented Jul 5, 2020

Pull Request for Issue # .

Summary of Changes

continue from #29984 (comment)
Searched for dragable and replaced by draggable

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Jul 5, 2020
@ceford
Copy link
Contributor

ceford commented Jul 6, 2020

I found 29 instances before patching and 18 after - all in media / system / js which I did not rebuild.


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

@hans2103
Copy link
Contributor Author

hans2103 commented Jul 6, 2020

@ceford That is due to the fact that all files in /media are in .gitignore
I cannot create a pull request for that one.


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

@zero-24
Copy link
Contributor

zero-24 commented Jul 6, 2020

The media folder should not have that any more after running npm. An other way to test that would be the pre build packages there npm should have rebuild them already.

@hans2103
Copy link
Contributor Author

hans2103 commented Jul 6, 2020

@zero-24 is there anything I have to change on my PR?

@zero-24
Copy link
Contributor

zero-24 commented Jul 6, 2020

@zero-24 is there anything I have to change on my PR?

No all is fine :)

@bonzani
Copy link

bonzani commented Jul 13, 2020

I have tested this item ✅ successfully on 1923fb5


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

1 similar comment
@Quy
Copy link
Contributor

Quy commented Jul 14, 2020

I have tested this item ✅ successfully on 1923fb5


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

@Quy
Copy link
Contributor

Quy commented Jul 14, 2020

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 14, 2020
@Quy
Copy link
Contributor

Quy commented Jul 15, 2020

Can you please update to include changes in #30094?

@hans2103
Copy link
Contributor Author

@Quy merged changes and found two new files with dragable instead of draggable.

@hans2103
Copy link
Contributor Author

I have added the merge in commit hans2103@89c852e
I added the changes of the two files in commit hans2103@e7490ea

They are in branch https://github.com/hans2103/joomla-cms/commits/feature/draggable-typo-fixes
But I don't see them in this GitHub issue.
Any idea why?

@Quy
Copy link
Contributor

Quy commented Jul 15, 2020

They have been added to your PR here. See e7490ea

Thank you!!!

@hans2103
Copy link
Contributor Author

@Quy yes! Now I see them too.

@richard67 richard67 merged commit 0e10650 into joomla:4.0-dev Jul 20, 2020
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jul 20, 2020
@richard67
Copy link
Member

Thanks!

@richard67 richard67 added this to the Joomla 4.0 milestone Jul 20, 2020
@hans2103 hans2103 deleted the feature/draggable-typo-fixes branch July 20, 2020 21:13
dgrammatiko added a commit to dgrammatiko/joomla-cms that referenced this pull request Jul 21, 2020
…outs

* '4.0-dev' of github.com:joomla/joomla-cms: (612 commits)
  [4.0] Smart Search: Fixing ordering, order direction and disabled button (joomla#29474)
  [4.0] Generate routed Modal links for iframes when not on the root (joomla#30007)
  [4.0] Get menu directly in com_tags menu route helper (joomla#30039)
  Remove collapse when resizing from mobile to desktop (joomla#30132)
  [4.0] Wrap component output in `main` element to make Cassiopeia more accessible (joomla#29870)
  [4.0] Webauthn gmp warning (joomla#29731)
  [4.0] Refactor to return early, remove if depths and throw NotAllowed (joomla#29694)
  [4.0] CLI help text (joomla#29811)
  Feature/draggable typo fixes (joomla#29987)
  [4.0] Removing unnecessary workaround in finder indexer (joomla#30037)
  [4.0] Optimizing Smart Search for larger content (joomla#30008)
  [4.0] Fix js ajax for pre update checker (joomla#29980)
  [4.0] Cassiopea: Fixing modals custom-select fields display (joomla#30097)
  [4.0][com_fields] Fix draggable sorting (joomla#30094)
  [4.0] Correct incorrect @return documentation (joomla#30092)
  [4.0] Menu items modal: adding missing filters (joomla#30087)
  short to long php open tags with echo (joomla#30089)
  Use new Toolbar (joomla#30085)
  [4.0] Center status/date created headers (joomla#29249)
  [4.0] Fix Cassiopea searchtools alignment in modals (joomla#30077)
  ...

# Conflicts:
#	administrator/components/com_templates/src/View/Template/HtmlView.php
#	installation/sql/postgresql/base.sql
#	libraries/src/Application/AdministratorApplication.php
#	libraries/src/Application/SiteApplication.php
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Replace "dragable" by "draggable"

* Searched for dragable and replaced by draggable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants