Skip to content

Commit

Permalink
fix: Do not depend on vue or vue-router for the public interface of `…
Browse files Browse the repository at this point in the history
…OC.Files.Router`

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
  • Loading branch information
susnux committed May 27, 2024
1 parent f251662 commit cfedbcb
Show file tree
Hide file tree
Showing 3 changed files with 39 additions and 131 deletions.
74 changes: 35 additions & 39 deletions lib/v27/OC.d.ts
Original file line number Diff line number Diff line change
@@ -1,46 +1,42 @@
import type { Route } from 'vue-router'

declare global {
namespace Nextcloud.v27 {
interface FilesRouter {
/**
* Trigger a route change on the files app
*
* @param path the url path, eg: '/trashbin?dir=/Deleted'
* @param replace replace the current history (default false)
* @see https://router.vuejs.org/guide/essentials/navigation.html#navigate-to-a-different-location
*/
goTo(path: string, replace?: boolean): Promise<Route>;

/**
* Trigger a route change on the files App
*
* @param name the route name
* @param params the route parameters
* @param query the url query parameters
* @param replace replace the current history
* @see https://router.vuejs.org/guide/essentials/navigation.html#navigate-to-a-different-location
*/
goToRoute(
name?: string,
params?: Record<string, string>,
query?: Record<string, string | (string | null)[] | null | undefined>,
replace?: boolean,
): Promise<Route>;
}
declare namespace Nextcloud.v27 {
interface FilesRouter {
/**
* Trigger a route change on the files app
*
* @param path the url path, eg: '/trashbin?dir=/Deleted'
* @param replace replace the current history (default false)
* @see https://v3.router.vuejs.org/guide/essentials/navigation.html
*/
goTo(path: string, replace?: boolean): Promise<void>;

/**
* Trigger a route change on the files App
*
* @param name the route name
* @param params the route parameters
* @param query the url query parameters
* @param replace replace the current history
* @see https://v3.router.vuejs.org/guide/essentials/navigation.html
*/
goToRoute(
name?: string,
params?: Record<string, string>,
query?: Record<string, string | (string | null)[] | null | undefined>,
replace?: boolean,
): Promise<void>;
}

interface OC extends Nextcloud.v26.OC {
interface OC extends Nextcloud.v26.OC {

}
}

interface OCP extends Omit<Nextcloud.v26.OCP, 'Files'> {
Files: {
Router: FilesRouter
}
interface OCP extends Omit<Nextcloud.v26.OCP, 'Files'> {
Files: {
Router: FilesRouter
}
}

interface WindowWithGlobals extends Nextcloud.Common.DayMonthConstants, Window {
interface WindowWithGlobals extends Nextcloud.Common.DayMonthConstants, Window {

}
}
}
}
92 changes: 3 additions & 89 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 1 addition & 3 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,7 @@
"url": "https://github.com/nextcloud/nextcloud-typings"
},
"dependencies": {
"@types/jquery": "3.5.16",
"vue": "^2.7.15",
"vue-router": "<4"
"@types/jquery": "3.5.16"
},
"devDependencies": {
"babel-jest": "^29.5.0",
Expand Down

0 comments on commit cfedbcb

Please sign in to comment.