Replace search term path with query
A search term is dynamic and not a pointer to a resource such as a channel as such it should be a query. For now the network as well as the channels are still in the path even though we should take them out of there as well (in the case we want a global / network search later on). As for now we can keep in as there is no such filter / facet yet.
This commit is contained in:
parent
be141bea65
commit
e36ae64c83
@ -127,7 +127,9 @@ export default {
|
||||
params: {
|
||||
uuid: this.network.uuid,
|
||||
target: this.channel.name,
|
||||
term: this.searchInput,
|
||||
},
|
||||
query: {
|
||||
q: this.searchInput,
|
||||
},
|
||||
});
|
||||
},
|
||||
|
@ -17,7 +17,7 @@
|
||||
<div class="header">
|
||||
<SidebarToggle />
|
||||
<span class="title"
|
||||
>Search results for "{{ $route.params.term }}" in
|
||||
>Search results for "{{ $route.query.q }}" in
|
||||
{{ $route.params.target }}</span
|
||||
>
|
||||
<span class="topic"></span>
|
||||
@ -148,7 +148,7 @@ export default {
|
||||
"$route.params.target"() {
|
||||
this.doSearch();
|
||||
},
|
||||
"$route.params.term"() {
|
||||
"$route.query.q"() {
|
||||
this.doSearch();
|
||||
},
|
||||
messages() {
|
||||
@ -195,7 +195,7 @@ export default {
|
||||
socket.emit("search", {
|
||||
networkUuid: this.$route.params.uuid,
|
||||
channelName: this.$route.params.target,
|
||||
searchTerm: this.$route.params.term,
|
||||
searchTerm: this.$route.query.q,
|
||||
offset: this.offset,
|
||||
});
|
||||
},
|
||||
@ -209,7 +209,7 @@ export default {
|
||||
socket.emit("search", {
|
||||
networkUuid: this.$route.params.uuid,
|
||||
channelName: this.$route.params.target,
|
||||
searchTerm: this.$route.params.term,
|
||||
searchTerm: this.$route.query.q,
|
||||
offset: this.offset + 1,
|
||||
});
|
||||
},
|
||||
|
@ -66,7 +66,7 @@ const router = new VueRouter({
|
||||
},
|
||||
{
|
||||
name: "SearchResults",
|
||||
path: "/search/:uuid/:target/:term",
|
||||
path: "/search/:uuid/:target",
|
||||
component: SearchResults,
|
||||
},
|
||||
],
|
||||
|
Loading…
Reference in New Issue
Block a user