FEATURE: Move route to /docs
After further discussion about the route, we landed on /docs as a better choice. This commit makes that change.
This commit is contained in:
parent
25ca793555
commit
14861e3321
|
@ -1,3 +1,3 @@
|
||||||
export default function() {
|
export default function() {
|
||||||
this.route("knowledgeExplorer", { path: "/e" });
|
this.route("knowledgeExplorer", { path: "/docs" });
|
||||||
}
|
}
|
||||||
|
|
|
@ -17,7 +17,7 @@ export default {
|
||||||
if (params.orderColumn) filters.push(`order=${params.orderColumn}`);
|
if (params.orderColumn) filters.push(`order=${params.orderColumn}`);
|
||||||
if (params.page) filters.push(`page=${params.page}`);
|
if (params.page) filters.push(`page=${params.page}`);
|
||||||
|
|
||||||
let promise = ajax(`/e.json?${filters.join("&")}`);
|
let promise = ajax(`/docs.json?${filters.join("&")}`);
|
||||||
|
|
||||||
if (params.selectedTopic) {
|
if (params.selectedTopic) {
|
||||||
promise = promise.then(data => {
|
promise = promise.then(data => {
|
||||||
|
|
|
@ -6,8 +6,8 @@ module ::KnowledgeExplorer
|
||||||
|
|
||||||
config.after_initialize do
|
config.after_initialize do
|
||||||
Discourse::Application.routes.append do
|
Discourse::Application.routes.append do
|
||||||
mount ::KnowledgeExplorer::Engine, at: '/e'
|
mount ::KnowledgeExplorer::Engine, at: '/docs'
|
||||||
get '/knowledge-explorer', to: redirect("/e")
|
get '/knowledge-explorer', to: redirect("/docs")
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -168,7 +168,7 @@ module KnowledgeExplorer
|
||||||
filters.push('page=1')
|
filters.push('page=1')
|
||||||
end
|
end
|
||||||
|
|
||||||
"/e.json?#{filters.join('&')}"
|
"/docs.json?#{filters.join('&')}"
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -18,7 +18,7 @@ describe KnowledgeExplorer::KnowledgeExplorerController do
|
||||||
describe 'knowledge explorer data' do
|
describe 'knowledge explorer data' do
|
||||||
context 'when any user' do
|
context 'when any user' do
|
||||||
it 'should return the right response' do
|
it 'should return the right response' do
|
||||||
get '/e.json'
|
get '/docs.json'
|
||||||
|
|
||||||
expect(response.status).to eq(200)
|
expect(response.status).to eq(200)
|
||||||
|
|
||||||
|
@ -41,7 +41,7 @@ describe KnowledgeExplorer::KnowledgeExplorerController do
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'should not show topics in private categories without permissions' do
|
it 'should not show topics in private categories without permissions' do
|
||||||
get '/e.json'
|
get '/docs.json'
|
||||||
|
|
||||||
json = JSON.parse(response.body)
|
json = JSON.parse(response.body)
|
||||||
topics = json['topics']['topic_list']['topics']
|
topics = json['topics']['topic_list']['topics']
|
||||||
|
@ -53,7 +53,7 @@ describe KnowledgeExplorer::KnowledgeExplorerController do
|
||||||
admin = Fabricate(:admin)
|
admin = Fabricate(:admin)
|
||||||
sign_in(admin)
|
sign_in(admin)
|
||||||
|
|
||||||
get '/e.json'
|
get '/docs.json'
|
||||||
|
|
||||||
json = JSON.parse(response.body)
|
json = JSON.parse(response.body)
|
||||||
topics = json['topics']['topic_list']['topics']
|
topics = json['topics']['topic_list']['topics']
|
||||||
|
@ -64,7 +64,7 @@ describe KnowledgeExplorer::KnowledgeExplorerController do
|
||||||
|
|
||||||
context 'when filtering by tag' do
|
context 'when filtering by tag' do
|
||||||
it 'should return a list filtered by tag' do
|
it 'should return a list filtered by tag' do
|
||||||
get '/e.json?tags=test'
|
get '/docs.json?tags=test'
|
||||||
|
|
||||||
expect(response.status).to eq(200)
|
expect(response.status).to eq(200)
|
||||||
|
|
||||||
|
@ -86,7 +86,7 @@ describe KnowledgeExplorer::KnowledgeExplorerController do
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'should return a list filtered by category' do
|
it 'should return a list filtered by category' do
|
||||||
get "/e.json?category=#{category2.id}"
|
get "/docs.json?category=#{category2.id}"
|
||||||
|
|
||||||
expect(response.status).to eq(200)
|
expect(response.status).to eq(200)
|
||||||
|
|
||||||
|
@ -116,7 +116,7 @@ describe KnowledgeExplorer::KnowledgeExplorerController do
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'should correctly filter topics' do
|
it 'should correctly filter topics' do
|
||||||
get "/e.json?search=banana"
|
get "/docs.json?search=banana"
|
||||||
|
|
||||||
expect(response.status).to eq(200)
|
expect(response.status).to eq(200)
|
||||||
|
|
||||||
|
@ -130,7 +130,7 @@ describe KnowledgeExplorer::KnowledgeExplorerController do
|
||||||
|
|
||||||
expect(topics.size).to eq(2)
|
expect(topics.size).to eq(2)
|
||||||
|
|
||||||
get "/e.json?search=walk"
|
get "/docs.json?search=walk"
|
||||||
|
|
||||||
json = JSON.parse(response.body)
|
json = JSON.parse(response.body)
|
||||||
topics = json['topics']['topic_list']['topics']
|
topics = json['topics']['topic_list']['topics']
|
||||||
|
|
Loading…
Reference in New Issue