750001 - [Firefox Flicks] Search by category, region or keyword is broken

Status: VERIFIED FIXED

(Websites :: Firefox Flicks, defect)

Description

[Santiago Hollmann [:nuno]]

Description
When you search by category, region or keyword and then you change to another page of the results search, the website shows all the results without filtering them.

Steps
1- Go to videos
2- Choose Latin America in region
3- Click on search
4- Go to the bottom and click on page number 2

Problem
When you change of page number, the website redirects to https://firefoxflicks.mozilla.org/en-US/recent?limit=&page=2 but it should redirect to https://firefoxflicks.mozilla.org/en-US/recent?category=all&region=latin_america&page=2

Comment 1

[[github robot]]

Commit pushed to develop at https://github.com/mozilla/firefox-flicks

https://github.com/mozilla/firefox-flicks/commit/0efa2b497bc031910ea59b4f882294b23aacd17e
Preserve search arguments between pages.

fix bug 750001

Comment 2

[Chris Beard]

This is still broken

Comment 3

[Christie Koehler [:ckoehler]]

Resolved doesn't mean the bugfix has been released. QA needs to verify the fix and then it will go into the next possible release. I will work on getting this verified and out as soon as possible.

Comment 4

[Chris More [:cmore]]

(In reply to Chris Beard from comment #2)
> This is still broken

As Christie mentioned, "resolved fixed" does not always mean it is live on the website. Since no developer has access to push changes to production, a QA engineer must first verify the fix. The workflow in bugzilla is set up that QA only verify items that have been set to "resolved fixed". If a bug is left "open", QA will not verify it and it will not be released on the public website. Once a QA engineer verifies the bug, it will ready to go out on the next available release executed by IT. 

I will let Christie update us when it has been verified on dev/stage and when IT can release the fix.

Comment 5

[raymond [:retornam] (inactive)]

qa-verified on dev 
https://firefoxflicks-dev.allizom.org/en-US/recent?category=thirty_spot&search=&limit=&region=all&page=2


waiting to get this on stage too

Comment 6

[raymond [:retornam] (inactive)]

qa-verified-stage https://firefoxflicks.allizom.org/en-US/recent?category=all&search=&limit=&region=america&page=2

Comment 7

[raymond [:retornam] (inactive)]

verified fixed https://firefoxflicks.mozilla.org/en-US/recent?category=all&search=&limit=&region=latin_america&page=3

Comment 8

[Chris More [:cmore]]

Thanks Raymond for getting this through QA phases today.

Comment 9

[raymond [:retornam] (inactive)]

You are welcome. Thanks to everyone that helped push this

Comment 10

[Santiago Hollmann [:nuno]]

Great guys! It's working perfectly :)