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.

Sign up for GitHub

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

Jump to bottom

DEV: Update bulk-invite logs and PM template #27057

Merged
merged 2 commits into from
May 17, 2024
Merged

DEV: Update bulk-invite logs and PM template #27057

merged 2 commits into from
May 17, 2024

Conversation

janzenisaac
Copy link
Contributor

@janzenisaac janzenisaac commented May 16, 2024 •
edited

Preview

Screenshot 2024-05-17 at 8 50 03 AM

New Logs

[2024-05-17 08:49:54 -0600] Invalid User Field 'backend name' for 'foobarbing@gmail.com'
[2024-05-17 08:49:54 -0600] Invalid Email 'test
[2024-05-17 08:49:54 -0600] Invalid Email 'this@$@**.com

@janzenisaac janzenisaac changed the title DEV: Update bulk-upload error messages and PM template DEV: Update bulk-invite error messages and PM template May 16, 2024
@janzenisaac janzenisaac changed the title DEV: Update bulk-invite error messages and PM template DEV: Update bulk-invite logs and PM template May 16, 2024
@github-actions github-actions bot added the i18n PRs which update English locale files or i18n related code label May 16, 2024
@janzenisaac
Copy link
Contributor Author

Will add tests, waiting for feedback on what data we want to include in the PR.

@janzenisaac janzenisaac force-pushed the update-bulk-upload-error-messages branch from 1e71699 to afcf4b9 Compare May 17, 2024 17:57
@janzenisaac janzenisaac marked this pull request as ready for review May 17, 2024 17:57
@janzenisaac janzenisaac merged commit ede0fa5 into main May 17, 2024
16 checks passed
@janzenisaac janzenisaac deleted the update-bulk-upload-error-messages branch May 17, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jbrw jbrw jbrw approved these changes

Assignees
No one assigned
Labels
i18n PRs which update English locale files or i18n related code
Milestone
No milestone
2 participants
@janzenisaac @jbrw