Skip to content

feat: custom users tab #44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 24 commits into
base: main
Choose a base branch
from
Open

feat: custom users tab #44

Show file tree
Hide file tree
Changes from 18 commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
7976aa7
wip: basic user form UI
brettkolodny Jul 3, 2025
13f88f8
feat: users form implementation
brettkolodny Jul 7, 2025
49be0fe
feat: add close button to user form
brettkolodny Jul 7, 2025
9a80a7e
feat: upload users
brettkolodny Jul 7, 2025
42971de
chore: refactor code to use consistent language of user vs. owner
brettkolodny Jul 8, 2025
897d214
feat: include user data in share
brettkolodny Jul 8, 2025
dd49328
fix: use theme responsive color for form elements
brettkolodny Jul 8, 2025
6b22290
fix: don't default user form to editing
brettkolodny Jul 8, 2025
ba5e9a4
fix: remove unused code
brettkolodny Jul 9, 2025
e789bc2
feat: show error dialog if users upload fails to parse
brettkolodny Jul 9, 2025
874d5c4
chore: remove console.error
brettkolodny Jul 9, 2025
9eb5646
Merge branch 'main' into brett/custom-users-tab
brettkolodny Jul 10, 2025
da6da72
fix: add prop
brettkolodny Jul 10, 2025
1269bd9
fix: add overflow
brettkolodny Jul 10, 2025
9f0cdfa
fix: remove forgotten HEAD
brettkolodny Jul 10, 2025
bdb06e7
fix: snippet name
brettkolodny Jul 10, 2025
eabedd5
Merge branch 'main' into brett/custom-users-tab
brettkolodny Jul 15, 2025
63185c7
Merge branch 'main' into brett/custom-users-tab
brettkolodny Jul 16, 2025
24470f3
chore: remove empty file
brettkolodny Jul 17, 2025
60c6a93
chore: add biome as a dev dependency
brettkolodny Jul 17, 2025
7cea52c
chore: update biome config to match saved version
brettkolodny Jul 17, 2025
d282a70
chore: remove newline
brettkolodny Jul 17, 2025
c1725f5
chore: reverse logic of user search
brettkolodny Jul 17, 2025
5229645
chore: remove unused biome config
brettkolodny Jul 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 16 additions & 27 deletions biome.jsonc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,13 @@
"vcs": {
"enabled": true,
"useIgnoreFile": true,
"clientKind": "git",
"root": ".."
"clientKind": "git"
},
"files": {
"ignore": [
"e2e/**/*Generated.ts",
"experimentalScannerIgnores": [
"src/client/gen/types.ts",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is what we want to use here. The docs say the new replacement for files.ignore is actually...

{
  "files": {
    "include": ["!src/client/gen/types.ts"]
  }
}

(note the ! in there)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to the new replacement but forgot to remove the old one 🤦

"pnpm-lock.yaml"

],
"ignoreUnknown": true
},
Expand All @@ -30,46 +30,35 @@
"level": "off"
},
"noParameterAssign": {
"level": "off"
"level": "off",
"options": {}
},
"useDefaultParameterLast": {
"level": "off"
},
"useSelfClosingElements": {
"level": "off"
"level": "off",
"options": {}
}
},
"suspicious": {
"noArrayIndexKey": {
"level": "off"
},
"noConsoleLog": {
"level": "error"
"noConsole": {
"level": "error",
"options": {
"allow": ["error"]
}
},
"noThenProperty": {
"level": "off"
}
},
"nursery": {
"useSortedClasses": "error",
"noRestrictedImports": {
"level": "error",
"options": {
"paths": {
"@mui/material": "Use @mui/material/<name> instead. See: https://material-ui.com/guides/minimizing-bundle-size/.",
"@mui/icons-material": "Use @mui/icons-material/<name> instead. See: https://material-ui.com/guides/minimizing-bundle-size/.",
"@mui/material/Avatar": "Use components/Avatar/Avatar instead.",
"@mui/material/Alert": "Use components/Alert/Alert instead.",
"@mui/material/Popover": "Use components/Popover/Popover instead.",
"@mui/material/Typography": "Use native HTML elements instead. Eg: <span>, <p>, <h1>, etc.",
"@mui/material/Box": "Use a <div> instead.",
"@mui/material/styles": "Import from @emotion/react instead.",
"lodash": "Use lodash/<name> instead."
}
}
}
"useSortedClasses": "error"
}
}
},
"$schema": "https://biomejs.dev/schemas/1.9.4/schema.json"
}
"$schema": "https://biomejs.dev/schemas/2.0.6/schema.json"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see a matching package.json version bump. do we not have biome as a dev dependency?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean by this? I've added biome a dev dependency and changed the schema URL to match.

}
2 changes: 2 additions & 0 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@
"@radix-ui/react-tabs": "^1.1.12",
"@radix-ui/react-tooltip": "^1.2.7",
"@tailwindcss/typography": "^0.5.16",
"@tanstack/react-form": "^1.12.4",
"@tanstack/valibot-form-adapter": "^0.42.1",
"@universal-middleware/core": "^0.4.7",
"@universal-middleware/hono": "^0.4.12",
"@vercel/blob": "^1.1.1",
Expand Down
94 changes: 92 additions & 2 deletions pnpm-lock.yaml
View file
Open in desktop

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

Loading