fix: handle owner but no name syntax (#2045)

This commit is contained in:
Garrett Delfosse
2022-06-06 10:38:38 -04:00
committed by GitHub
parent 77d068215a
commit 3be3bc261e
2 changed files with 14 additions and 10 deletions
+1
View File
@@ -71,6 +71,7 @@ describe("util > workspace", () => {
["owner:me", { owner: "me" }],
["owner:me owner:me2", { owner: "me" }],
["me/dev", { owner: "me", name: "dev" }],
["me/", { owner: "me" }],
[" key:val owner:me ", { owner: "me" }],
])(`query=%p, filter=%p`, (query, filter) => {
expect(workspaceQueryToFilter(query)).toEqual(filter)
+13 -10
View File
@@ -218,24 +218,27 @@ export const workspaceQueryToFilter = (query?: string): TypesGen.WorkspaceFilter
const parts = preparedQuery.split(" ")
for (const part of parts) {
const [key, val] = part.split(":")
if (key && val) {
if (key === "owner") {
return {
owner: val,
if (part.includes(":")) {
const [key, val] = part.split(":")
if (key && val) {
if (key === "owner") {
return {
owner: val,
}
}
// skip invalid key pairs
continue
}
// skip invalid key pairs
continue
}
const [username, name] = part.split("/")
if (username && name) {
if (part.includes("/")) {
const [username, name] = part.split("/")
return {
owner: username,
name: name,
name: name === "" ? undefined : name,
}
}
return {
name: part,
}