Merge pull request #1037 from ssiyad/feat/article_title_slug

feat: article title slug in url
frappe · Mar 9, 2023 · 1402817 · 1402817
2 parents 033aac8 + 0bec23a
commit 1402817
Show file tree

Hide file tree

Showing 8 changed files with 940 additions and 948 deletions.
diff --git a/desk/src/components/global/kb/ArticleMiniList.vue b/desk/src/components/global/kb/ArticleMiniList.vue
@@ -32,7 +32,7 @@
  $router.push({
  path: `/${
  $route.meta.editable ? 'frappedesk' : 'support'
- }/kb/articles/${element.name}`,
+ }/kb/articles/${element.name}/${element.title_slug}`,
  });
  }
  "
@@ -137,14 +137,12 @@ export default {
  resources: {
  articles() {
  const filters = { category: this.categoryId, status: "Published" };
- const fields = ["name", "title", "idx", "status"];
 
  return {
  cache: ["Articles", this.categoryId, "published"],
  method: "frappedesk.api.kb.get_articles",
  params: {
  filters,
- fields,
  limit: 999,
  order_by: "idx",
  },