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

When Adding Yourself To A Team That You Created, Accepting Invite Causes Error #3110

Closed
weilandia opened this issue Sep 16, 2016 · 21 comments
Closed

Comments

@weilandia
Copy link
Sponsor

weilandia commented Sep 16, 2016

When a user goes to /teams/new and subsequently adds themselves to the team invite list and creates the team, the user gets an invite to the team that blows up upon accepting the the invite.

Potential solution: If a user adds themselves to a team that they are creating, they should not get an invite.

screen shot 2016-09-16 at 12 34 31 pm

@weilandia
Copy link
Sponsor Author

screen shot 2016-09-16 at 12 34 37 pm

@weilandia
Copy link
Sponsor Author

I can work on this if ya'll want. Lmk :)

@mhelmetag
Copy link

Go for it! Looking forward to the PR and seeing what approach you took.

@bernardoamc
Copy link

Thank you for noticing this, don't hesitate to ping me if you have questions! ❤️

@jtigger jtigger added the bug label Oct 2, 2016
@aaron-junot
Copy link

Also, if you created the group, then you refuse the invite, it doesn't take you away from the group.

@weilandia Are you still working this? If not, we can go ahead and take it over.

@MohanL
Copy link

MohanL commented Oct 11, 2016

@aaron-suarez, do you mind if I join your team for this bug ?

@aaron-junot
Copy link

I have no problem with that. @nellshamrell is our team leader, so I'll let you coordinate with her about the coordination details 🙂

@weilandia
Copy link
Sponsor Author

@aaron-suarez go for it!

@MohanL
Copy link

MohanL commented Oct 13, 2016

@aaron-suarez @nellshamrell
how are you guys going to do this ? How can I join the team ?

@aaron-junot
Copy link

Sorry if there was a lack of communication for a few days there. We typically have a conference call on Saturdays but this last one was especially inconvenient for several of us and no one was really able to make it. As far as I'm concerned, if you find a solution to this between now and next Sat, feel free to submit the pull request and we'll pick a different issue for that week. If you don't find a solution, I'll send you an invite to our call and we can all collaborate together. @nellshamrell let me know if you have any issues with that

@MohanL
Copy link

MohanL commented Oct 19, 2016

@weilandia @aaron-suarez
can you guys try to replicate the error ?
I have tried and it seems like when I invite myself, I don't get any notifications.

@aaron-junot
Copy link

aaron-junot commented Oct 19, 2016

I did replicate the error:

screenshot from 2016-10-18 23-55-18

After inputting the above, hit "save" at the bottom (obviously with your username) and you'll get this:

screenshot from 2016-10-18 23-55-34

@MohanL
Copy link

MohanL commented Oct 19, 2016

@aaron-suarez please do invite me for the conference call, Thank you

@nellshamrell
Copy link

@aaron-suarez The more the merrier!

@Insti
Copy link

Insti commented Oct 26, 2016

Is getting an invite for your own group even a good idea?
I expected I would just create the group and be a member without any invitation and was surprised when one showed up in my notifications.

@aaron-junot
Copy link

@Insti I agree that there probably should be no invitation given to the group creator. This would solve both issues discussed in the thread. We'll make it so that the group creator can't receive an invitation to their own group, but are instead automatically added as a manager.

@MohanL
Copy link

MohanL commented Oct 26, 2016

@aaron-suarez
we could probably do a check on the team's member set to decide whether it is valid to add the user. what do you think ? Also, we could have notifications saying that someone is already a member of the team if the invitation is not valid. What do you think ?

@aaron-junot
Copy link

That sounds good. By the way, have you been getting any of the emails I've sent to contact@mohanl.com ?

@MohanL
Copy link

MohanL commented Oct 27, 2016

@aaron-suarez Yes, I just saw it. sorry about the late reply.

@MohanL MohanL mentioned this issue Oct 29, 2016
3 tasks
@lizvdk
Copy link

lizvdk commented Dec 12, 2016

Does anyone want to take this on? It looks like #3219 is not moving forward.

@bernardoamc
Copy link

Will take a look when I get home, thanks for bumping it up @lizvdk!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants