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

As inviter, I want the ability to set the status of invited people directly when creating the appointment #333

Closed
FloG111 opened this issue Aug 22, 2022 · 8 comments · Fixed by #349
Assignees

Comments

@FloG111
Copy link

FloG111 commented Aug 22, 2022

image

In the red frame, I can choose individual members of the space and can select the status for them.
In the green frame, I can select all space members, but can't select the status.

I want this choice, because of organisatial topics. I want the team members (in a sport group) to be there in training and just to say no, in case when they won't have time.

@yurabakhtin
Copy link
Collaborator

@luke- Currently the text is "[ ] Invite all Space members", i.e. the members are added with status "Invite".

What do you think if we will add the status selector?

add-status

and myabe the word "Invite" should be changed to "Add"?

@luke-
Copy link
Contributor

luke- commented Dec 21, 2022

@yurabakhtin This would be a good solution and should cover all possible use cases. .

"Invited" should be the default value. What would then be the difference between "Undecided" and "Invited"? We might then remove "Undecided".

The text change would then be necessary

@FloG111
Copy link
Author

FloG111 commented Dec 21, 2022

I like the idea :)
For me, "attending" and "invited" is super. Perhaps for other people "declined" should be added to.

@yurabakhtin
Copy link
Collaborator

@luke-

"Invited" should be the default value. What would then be the difference between "Undecided" and "Invited"? We might then remove "Undecided".

Do you mean to remove the "Undecided" only from new selector of the checkbox "Invite all Space members"?
Maybe we should remove the option "Undecided" from the first selector as well, because it seems no reason to add users with such status by default.

The text change would then be necessary

"[ ] Add all Space members with status <selector>" - ok?

@luke-
Copy link
Contributor

luke- commented Dec 21, 2022

That's right, have the selector removed above as well.
Yes sounds good.

@Semir1212 Wording ok?

@Semir1212
Copy link
Contributor

@yurabakhtin @luke- Text is fine

@yurabakhtin
Copy link
Collaborator

@luke- @FloG111 Done in PR #349

Be default both status selectors are "Invited", but I tested this case:

test_new_status

@FloG111
Copy link
Author

FloG111 commented Dec 23, 2022

perfect, thanks a lot :)

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

Successfully merging a pull request may close this issue.

4 participants