Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix error from PR #12543 #21233

Closed
wants to merge 1 commit into from
Closed

Fix error from PR #12543 #21233

wants to merge 1 commit into from

Conversation

Tchangue
Copy link

Summary of Changes

The PR here #12543 generates an JS error, which is already in the system. This PR fix the error.

Testing Instructions

Apply the mentioned PR => JS error appears, apply patch => error disappear.

Expected result

No JS error

Actual result

image

@infograf768
Copy link
Member

infograf768 commented Jul 23, 2018

@Tchangue

One cannot add a PR to correct an error in a PR which is not merged yet.

In this case, it is better to correct the original PR or IF the original coder has disappeared, redo a full PR containing the whole code and all specific testing instructions, then ask Bugsquad to close the original PR.


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

@infograf768
Copy link
Member

As far as I can read in #12543
the new PR for that error is #12544

@roland-d
Copy link
Contributor

I am going to close this PR as #12543 is for testing purposes only. Thank you.

@roland-d roland-d closed this Jul 24, 2018
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.

4 participants