Refactor: move capture page to /captures #1804
Conversation
| @@ -80,7 +80,7 @@ function FeaturedTreesSlider({ trees, size = null, isMobile, link }) { | |||
| > | |||
| {trees.map((tree) => ( | |||
| <Link | |||
| href={link ? link(tree) : `/v2/captures/${tree.id}`} | |||
| href={link ? link(tree) : `/captures/${tree.id}`} | |||
There was a problem hiding this comment.
@dadiorchen this is what you're looking for?
cypress integration test is not added yet. just checking if i'm on the right path.
There was a problem hiding this comment.
@yunchipang yes, and to finish this, we also need a integration test for this: /capture page
There was a problem hiding this comment.
@yunchipang this is the right way to go, I guess you passed it on your local env, but on the ci it's broken:
https://github.com/Greenstand/treetracker-web-map-client/actions/runs/11695766183/job/32571757532?pr=1804#step:3:147
can you take a look
| clearNocks(); | ||
| }); | ||
|
|
||
| describe('Capture page', () => { |
There was a problem hiding this comment.
@dadiorchen cypress integration test for the capture page has been added and passed. could you kindly take a look? i have requested another review from you. thanks!
Description
Fixes #1768
Type of change
Screenshots
How Has This Been Tested?
Checklist: