-
Notifications
You must be signed in to change notification settings - Fork 2
Queue Invites for Draft Processes #707
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
Open
scazan
wants to merge
24
commits into
dev
Choose a base branch
from
invite-queue
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+371
−93
Open
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
31b908d
feat: [US-002] - Suppress email sending for invites to draft processes
scazan c874b74
feat: [US-003] - Send queued invite emails on process publish
scazan 8356de5
feat: [US-004] - Update Inngest workflow to mark invites as notified
scazan 59b8229
feat: [US-005] - Hide unnotified invites from invitee personal invite…
scazan 67ab7a3
feat: [US-006] - Show notified status in admin participant list
scazan 70f9201
fix: remove premature notified flag update in updateDecisionInstance
scazan 91f8f92
Bypass draft queueing for admins
scazan e9ec222
Update tests
scazan 8f3a539
Use proper color for pending
scazan 87d32ec
Add in banner
scazan 737f0e3
Add in alert to top of invites for non-notified
scazan 7e7d514
Pull in AlertBanner from dev and rebase
scazan 927439b
Switch to notified_at
scazan 2e02c03
pluralize tab labels
scazan 973d0c2
Update positioning of AlertBanners
scazan 67e23d4
Show modal in the case of pending notifications
scazan cfb022a
Show how many will be notified
scazan c5ada42
Use proper isDraft check
scazan dc0d384
Translations and better component
scazan 3bfb3bd
Simpler interfaces
scazan ce66f7b
Remove non-null assertion
scazan 2c16109
Pull out invite email message
scazan e20e99f
Extract function
scazan ac0f324
Remove non-null assertion
scazan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to note this is a temporary thing until guided UX is in place