Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Query parameter in /video page #33

Open
bunnyxt opened this issue Sep 18, 2020 · 3 comments
Open

Query parameter in /video page #33

bunnyxt opened this issue Sep 18, 2020 · 3 comments
Assignees
@bunnyxt
Labels
enhancement New feature or request refactor Code refactor, for better performance or easier to maintain

Comments

@bunnyxt
Copy link
Owner

bunnyxt commented Sep 18, 2020

Currently, whether user using what filter to query videos, the url is still /video, not including filter parameters for example, /video?start_ts=1359631780&tid=30&order_by=view&desc=1, which is quite not good for sharing and reloading. Need update using vue-router this.$route.query.your_params.

https://stackoverflow.com/questions/35914069/how-can-i-get-query-parameters-from-a-url-in-vue-js
@bunnyxt bunnyxt added the enhancement New feature or request label Sep 18, 2020
@bunnyxt bunnyxt self-assigned this Sep 18, 2020
@bunnyxt
Copy link
Owner Author

bunnyxt commented Sep 18, 2020

Try apply it in /rank first, then to all pages with list.

@bunnyxt
Copy link
Owner Author

bunnyxt commented Sep 29, 2020

finish trial in /rank

@bunnyxt bunnyxt added the refactor Code refactor, for better performance or easier to maintain label Oct 11, 2020
@bunnyxt
Copy link
Owner Author

bunnyxt commented Jun 15, 2021

#57 /video done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bunnyxt bunnyxt

Labels
enhancement New feature or request refactor Code refactor, for better performance or easier to maintain
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bunnyxt