Skip to content
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

Make form for adopting a project a ChoiceField #4777

Closed
dojutsu-user opened this issue Oct 18, 2018 · 10 comments
Closed

Make form for adopting a project a ChoiceField #4777

dojutsu-user opened this issue Oct 18, 2018 · 10 comments
Labels
Design Design or UX/UI related Improvement Minor improvement to code

Comments

@dojutsu-user
Copy link
Member

This is the form for adopting projects for Gold Members.

screenshot from 2018-10-18 11-29-40

Currently the input field does not specify any idea about what to enter.
Although, the project's slug is need to be enter, but it is written nowhere which creates confusion and the user ends up entering the project's name which gives an error.

This issue was discussed in the comments #427135794, #427137725 and #427139109

@dojutsu-user
Copy link
Member Author

@stsewd Will adding the help_text would be enough?

screenshot from 2018-10-18 12-09-43

@humitos
Copy link
Member

humitos commented Oct 18, 2018

That field should probably be a ChoiceField where the user can only select the available options (I'm assuming that the User can only adopt projects where he is a maintainer --but I'm not 100% sure, you will need to check the code to know that)

@dojutsu-user
Copy link
Member Author

@humitos
Copy link
Member

humitos commented Oct 18, 2018

Mmm... OK. It doesn't make too much sense to me, since it's a weird case to want to support a project that you are not involved to.

So, if we want to keep this behavior, my solution of the ChoiceField doesn't fit here.

Another different approach, that we don't have in any other place I think, would be to use a field with autocomplete --not sure if we want that, though. Probably, no.

I'd say that for now, the help_text is the first step, but it doesn't solve the whole problem anyway.

@dojutsu-user
Copy link
Member Author

@humitos yeahh, help_text won't solve whole problem, but it atleast gives the idea/hint that the project's name is not to be entered here.

@ericholscher
Copy link
Member

I'm +1 on adding it as a ChoiceField -- I think the current implementation wasn't a design decision. If someone needs to adopt a project they don't own, we can handle that as a support issue.

@ericholscher ericholscher reopened this Oct 31, 2018
@ericholscher ericholscher changed the title Add a label/help text in the form for adopting a project Make form for adopting a project a ChoiceField Oct 31, 2018
@dojutsu-user
Copy link
Member Author

@ericholscher So, can i start working on this?

@ericholscher
Copy link
Member

Sure!

@dojutsu-user
Copy link
Member Author

@ericholscher
I have made a PR, can you please review it.

@ericholscher
Copy link
Member

Fixed in #4841

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design Design or UX/UI related Improvement Minor improvement to code
Projects
None yet
Development

No branches or pull requests

4 participants