-
-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
Offer a way to edit streams/role details for invitations #22099
Comments
This is clearly a thing we'll want to support. It's probably most relevant for reusable invitation links, just because those are something one would want to maintain over time. It's likely a moderately complex project, since we don't currently send the data on what streams are associated with an invite to clients, but potentially fairly straightforward, especially if we can reuse the component for selecting which streams to select from the existing invitation UI. |
Hello @zulip/server-onboarding, @zulip/server-settings members, this issue was labeled with the "area: settings (admin/org)", "area: invitations" labels, so you may want to check it out! |
@zulipbot claim |
Welcome to Zulip, @bigBrain1901! We just sent you an invite to collaborate on this repository at https://github.com/zulip/zulip/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip! Here's some tips to get you off to a good start:
As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site. See you on the other side (that is, the pull request side)! |
That general approach sounds reasonable to me in terms of the user experience. |
One doubt, what is the reason for keeping back the data on associated streams from clients? Otherwise, we could consider a new endpoint to fetch this data on a per-invite basis like - This could be favourable if there are -
|
I think there's no reason we couldn't have I don't see any reason this change would be expensive if done with a proper bulk query. It might also be reasonable to add a |
The Case Story
A screencast of the expected behaviourscreen-recorder-wed-mar-08-2023-23-06-20.webmRequestPlease let me know if this is the desired behaviour, and any changes. |
Thanks! The "Edit" button should be a pencil, as in all the other menus. We may want to change the other buttons there to icons as well. As a note for next time, please include screenshots of key screens any time you're requesting feedback, as they are easier to review than a video. (Also posting a video can be helpful as well in case there's a question about the interactions.) |
Perfect, so I have a commit that adds this feature. But I haven't written tests for it yet. Shall I create a new Draft PR for an initial review of the UI/UX, and then proceed to write tests if that review passes? |
@bigBrain1901 You can always create a draft PR if that's helpful for your workflow, but your work will generally be reviewed when it's complete (including tests). |
The PR is now ready for review, including tests. (I messed up a rebase in the previous PR, hence this new one 👼 ) |
This commit enables admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit enables admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit enables admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit enables admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit enables admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit enables admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit enables admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit enables admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes part of zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes part of zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit adds frontend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes part of zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes part of zulip#22099 Signed-off-by: Ishaan Singh <asishaan.191cs124@nitk.edu.in>
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. additionally, Document "/invites/multiuse/{invite_id}" endpoint. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. additionally, Document "/invites/multiuse/{invite_id}" endpoint. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. additionally, Document "/invites/multiuse/{invite_id}" endpoint. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. additionally, Document "/invites/multiuse/{invite_id}" endpoint. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. additionally, Document PATCH "/invites/multiuse/{invite_id}" endpoint. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. additionally, Document PATCH "/invites/multiuse/{invite_id}" endpoint. Fixes zulip#22099.
This commit adds backend changes required for admins to edit multiuse invites with respect to the role and associated streams. Corresponding tests and API changelog updates have been added. additionally, Document PATCH "/invites/multiuse/{invite_id}" endpoint. Fixes zulip#22099.
Request: Be able to modify pending invites.
Reason: Sometime we forget to add a channel to an invite and it would make it very convenient if we could modify the pending invite instead of waiting for the user to join or cancel/recreate the invite.
The text was updated successfully, but these errors were encountered: