-
Notifications
You must be signed in to change notification settings - Fork 886
feat(setup): organization is not longer needed #4707
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
Conversation
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.
Thanks for taking a look at this one! It seems to me that the API is asking we send down an organization. You can see where we send down data of this type in api.ts
.
Not sure what needs to be altered server-side here, but we'll need to regenerate those types after a change is made so that lint stops complaining.
This Pull Request is becoming stale. In order to minimize WIP, prevent merge conflicts and keep the tracker readable, I'm going close to this PR in 3 days if there isn't more activity. |
b4f13c5
to
a2eaba1
Compare
a2eaba1
to
888c43a
Compare
I went back in there and removed all backend references. Would appreciate a second set of eyes on this one as first time making backend+frontend changes in the project. |
{...getFieldHelpers("organization")} | ||
onChange={onChangeTrimmed(form)} | ||
autoFocus | ||
fullWidth |
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.
It might be nice to give autoFocus
to the next field so that users can start typing without clicking into the form.
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.
FE ✅
Resolves (in theory) #4697