-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat/submit website #42
Conversation
…t with a few bugs
…gnin via web3auth through it
Two things have come to mind:
|
… signedIn from AuthContext
I'd prefer to show good error messages under the relevant input element than disabling the button
Agreed, we should require this. I don't think we should provide a default value though @ChialiT please weigh in on these^^ Some requests before approval:
👏🏽 Great stuff so far, glad to see this transformed from design to usable UI! |
Took care of those items we discussed and your requests before merging @thebeyondr |
@Ian-Bright looks good, discovered a few things
|
Ok disabled the slider when creating a tag. The sign in bugs I cannot reproduce. I've had something similar before on an earlier branch where I needed to refresh. Are there any steps to reproduce on your end @thebeyondr ? |
Resolves:
#25
#33
Redesign of the submit website page. User can input a site url, description (title), and tags before submitting a video. Users also have the ability to add new tags through this page