Skip to content

Conversation

@joomdonation
Copy link
Contributor

Pull Request for Issue #33891.

Summary of Changes

Append client_id to the redirection link to solve issue ##33891

Testing Instructions

  1. Install Joomla 4 nightly build
  2. Look at [4.0] Module: After status-change on admin-module screen switch to "Site" #33891 , confirm the issue
  3. Apply patch, confirm the issue sorted.

@ghost
Copy link

ghost commented May 15, 2021

I have tested this item ✅ successfully on 8239fd3


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

@sandewt
Copy link
Contributor

sandewt commented May 15, 2021

I can NOT confirm this issue in Windows 10, browsers: Google Chrome, MS Edge and Firefox.

Version: Joomla! 4.0.0-beta8-dev Development

@joomdonation
Copy link
Contributor Author

@sandewt You need to update to Joomla 4 nightly build to see this issue.

@alikon
Copy link
Contributor

alikon commented May 15, 2021

I have tested this item ✅ successfully on 8239fd3


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

@alikon
Copy link
Contributor

alikon commented May 15, 2021

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 15, 2021
@sandewt
Copy link
Contributor

sandewt commented May 15, 2021

@sandewt You need to update to Joomla 4 nightly build to see this issue.

@joomdonation, Indeed

@alikon
Copy link
Contributor

alikon commented May 15, 2021

just for curiosity what pr did the regression ? @joomdonation

@joomdonation
Copy link
Contributor Author

@alikon Could not be sure 100% as I haven't checked it yet. It is very likely it from my other PR #33763 . Hope nothing else is broken now.

@HLeithner HLeithner merged commit 1173c1b into joomla:4.0-dev May 15, 2021
@HLeithner
Copy link
Member

thanks

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 15, 2021
@HLeithner HLeithner added this to the Joomla 4.0 milestone May 15, 2021
@joomdonation joomdonation deleted the fix_redirect_in_modules_management branch May 15, 2021 14:28
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.

5 participants