Replace the url on the search results with urls that redirect through /link?url=blah

ucsb-cs56-w20 / mapache-search

https://mapache-search-qa.herokuapp.com/

2 stars 3 forks source link

Closed darraghjburke closed 4 years ago

darraghjburke commented 4 years ago

[x] We want the links on the search results page to point to /link?... rather than the URL of the link
[x] ~~We want to create a test that checks that the link on the search result page points to the correct URL~~ MOVED TO ISSUE #197

tahereh-lab commented 4 years ago

we can try to chang result.html

tahereh-lab commented 4 years ago

change this line in result.html

DubiousLatchkey commented 4 years ago

I just realized we did this as part of Issue #155, so I've moved this to under review as well and linked that PR here.

pconrad commented 4 years ago

I'm factoring out the creation of a test into a separate issue so that we can keep the team moving forward.

pconrad commented 4 years ago

Fixed in PR #190