-
Notifications
You must be signed in to change notification settings - Fork 86
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.
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
Fixes #61: Added an option to create user on the dashboard's "User" section #73
Conversation
@kabir276 is attempting to deploy a commit to the Stack Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I get 404 API key not found error when I try to create a user on the dashboard. Did it work for you?
|
||
export default function PageClient() { | ||
const stackAdminApp = useAdminApp(); | ||
const allUsers = stackAdminApp.useServerUsers(); | ||
const [addUserOpen, setAddUserOpen] = React.useState(false); | ||
const [showNotifyPasswordDialog, setShowNotifyPasswordDialog] = React.useState(false); | ||
const [currentUserPassword, setcurrentUserPassword] = React.useState(""); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
setcurrent...
-> setCurrent...
onClick: handlePasswordNotificationClose, | ||
}} | ||
open={showNotifyPasswordDialog} | ||
onClose={handlePasswordNotificationClose} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eslint not passing: you should use runAsynchronouslyWithAlert
, check out the other occurrence. Also please add the eslint plugin to vscode so the errors will be highlighted
emailVerificationRedirectUrl, | ||
null, | ||
); | ||
if (!errorCode) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a promise, you need to add await
Yes, I was getting the same issue but I explicitly commented that “if statement” causing the issue and asked @N2D4 about it. He asked me to make a PR and he’ll take a look at it. |
The PR seems to be inactive |
Fixes Issue #61
Description
I've added an option to create users on the dashboard's "Users" section. I've created a pull request for your review. Please let me know if this is what you envisioned. If not, I'm happy to make further adjustments.
Changes Made
Updated users>page.client.tsx , lib>stack-app.ts , Interface>clientInterface.ts to include the button to create users.
Additional Notes
No new dependencies were added.