Skip to content

[4.0] Fix clicking on title to edit a field group#26424

Merged
wilsonge merged 2 commits intojoomla:4.0-devfrom
wilsonge:bugfix/fix-editing-field-group
Sep 28, 2019
Merged

[4.0] Fix clicking on title to edit a field group#26424
wilsonge merged 2 commits intojoomla:4.0-devfrom
wilsonge:bugfix/fix-editing-field-group

Conversation

@wilsonge
Copy link
Contributor

Summary of Changes

Fixes a bug introduced in #24801 where you can't edit a field group by clicking on it's title

Testing Instructions

Create a field group, close it, then try to edit it by clicking the link on it's title

Expected result

The field group opens to edit

Actual result

Error

Documentation Changes Required

None

@wilsonge wilsonge changed the title Fix clicking on title to edit a field group [4.0] Fix clicking on title to edit a field group Sep 28, 2019
@Quy Quy mentioned this pull request Sep 28, 2019
@Quy
Copy link
Contributor

Quy commented Sep 28, 2019

I have tested this item ✅ successfully on 120fd8f


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

@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on c9c4a5b


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

@SharkyKZ
Copy link
Contributor

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Sep 28, 2019
@wilsonge wilsonge merged commit 67ce5d5 into joomla:4.0-dev Sep 28, 2019
@wilsonge
Copy link
Contributor Author

Thanks!

@wilsonge wilsonge deleted the bugfix/fix-editing-field-group branch September 28, 2019 20:24
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 28, 2019
@wilsonge wilsonge added this to the Joomla 4.0 milestone Sep 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

Comments