? ? Failure

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
8 May 2016

Pull Request for Improvement.

Summary of Changes

Following #10217, this PR adds a new status code filter to com_redirect.

image

Testing Instructions

  1. Apply patch on latest staging
  2. Go to Components -> Redirect and test the new HTTP status code filter.
avatar andrepereiradasilva andrepereiradasilva - open - 8 May 2016
avatar andrepereiradasilva andrepereiradasilva - change - 8 May 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 May 2016
Labels Added: ? ?
avatar andrepereiradasilva andrepereiradasilva - change - 8 May 2016
The description was changed
avatar brianteeman brianteeman - change - 8 May 2016
Category Components Language & Strings UI/UX
avatar brianteeman brianteeman - change - 8 May 2016
Labels
avatar andrepereiradasilva andrepereiradasilva - change - 8 May 2016
The description was changed
avatar brianteeman brianteeman - test_item - 8 May 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 8 May 2016

I have tested this item :white_check_mark: successfully on 7bb54c7


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

avatar zero-24 zero-24 - test_item - 8 May 2016 - Tested successfully
avatar zero-24
zero-24 - comment - 8 May 2016

I have tested this item :white_check_mark: successfully on 7bb54c7

it works


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

avatar zero-24 zero-24 - change - 8 May 2016
Status Pending Needs Review
avatar zero-24
zero-24 - comment - 8 May 2016

I have movend it to needs review so the PLT can decide to add this filter or not. as per default this option is disabled but we can't disable or enable the filter based on that.


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 8 May 2016

the option in new/edit is (some nonsense of advanced mode IMHO).
But the status columns is always there.

avatar zero-24
zero-24 - comment - 8 May 2016

i can't see it here @andrepereiradasilva

redirects

avatar andrepereiradasilva
andrepereiradasilva - comment - 8 May 2016

arwe you using latest staging?

avatar andrepereiradasilva
andrepereiradasilva - comment - 8 May 2016

see here bb1911e

avatar Kubik-Rubik
Kubik-Rubik - comment - 8 May 2016

Good change, thank you @andrepereiradasilva!

avatar Kubik-Rubik Kubik-Rubik - change - 8 May 2016
Status Needs Review Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-05-08 16:07:00
Closed_By Kubik-Rubik
avatar Kubik-Rubik Kubik-Rubik - reference | 679ca77 - 8 May 16
avatar Kubik-Rubik Kubik-Rubik - merge - 8 May 2016
avatar Kubik-Rubik Kubik-Rubik - close - 8 May 2016
avatar andrepereiradasilva andrepereiradasilva - head_ref_deleted - 8 May 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 8 May 2016

thansk for testing, comments and merging!

avatar brianteeman brianteeman - change - 10 May 2016
Milestone Added:

Add a Comment

Login with GitHub to post a comment