Skip to content
This repository has been archived by the owner on Dec 20, 2023. It is now read-only.

Nav url params #33

Merged
merged 9 commits into from
Mar 26, 2022
Merged

Nav url params #33

merged 9 commits into from
Mar 26, 2022

Conversation

namantaneja167
Copy link
Contributor

Closes #22

Description

Clicking on different test cases generates a unique URL that can be shared easily.

Additional Information

@nehagup
Copy link
Member

nehagup commented Mar 26, 2022

Looks like there's underline css added to the test-list apps list component @namantaneja167

Screenshot 2022-03-27 at 12 12 27 AM

@nehagup
Copy link
Member

nehagup commented Mar 26, 2022

Looks like when we close the test-case the url params doesn't change and tcId doesn't get removed. @namantaneja167

Screenshot 2022-03-27 at 12 14 26 AM

@nehagup
Copy link
Member

nehagup commented Mar 26, 2022

Rest LGTM! 🎉🌟

@nehagup
Copy link
Member

nehagup commented Mar 26, 2022

LGTM

@nehagup nehagup merged commit 10a1a07 into keploy:main Mar 26, 2022
@nehagup
Copy link
Member

nehagup commented Mar 27, 2022

This was referenced Mar 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature]: Add test case and app ids as url params on the test list page
2 participants