- Issue created by @fwust
- Assigned to sourabhsisodia_
- Merge request !3Issue #3442639: Corrected previous page href to absolute url of page in the pager. → (Merged) created by sourabhsisodia_
- Issue was unassigned.
- Status changed to Needs review
4 months ago 11:43am 8 July 2024 @fwust The patch looks good , created a MR for it ,Tested the MR . Attaching a Screenshot for reference.
-
sourabhsisodia_ →
committed ecbdf07e on 1.0.x
Issue #3442639: Corrected previous page href to absolute url of page in...
-
sourabhsisodia_ →
committed ecbdf07e on 1.0.x
- Status changed to RTBC
3 months ago 11:16am 29 August 2024 Hello,
I have followed the below steps to review,- Installed the module, used the frontpage view where I enabled Ajax.
- Went to page 2 and checked the prev button href which had /views/ajax.
- Applied the MR!3, which applied cleanly.
- Rechecked by going to page 3, checked the prev button href which had /node (correct href of frontpage).
Have included the before and after ss for reference.
Hence, moving to RTBC.
Thanks :)Hi @_cosmos_,
Thanks for merging the MR.I noticed that I wasn't credited for resolving this issue. I'm not sure why that happened. Is there something I'm missing?