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

Print number and error for failing contact method validation #568

Merged
merged 1 commit into from
Apr 5, 2023

Conversation

zecke
Copy link
Contributor

@zecke zecke commented Sep 9, 2022

The validation is great but sometimes it is helpful to see what the input was and the error that as produced. Print the input and the error when available.

Copy link
Contributor

@drastawi drastawi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very helpful. lgtm.

@zecke zecke requested a review from drastawi March 27, 2023 09:40
@imjaroiswebdev imjaroiswebdev requested review from imjaroiswebdev and removed request for drastawi April 5, 2023 16:41
The validation is great but sometimes it is helpful to see what
the input was and the error that as produced. Print the input
and the error when available.
Copy link
Contributor

@imjaroiswebdev imjaroiswebdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank for this valuable addition @zecke, because your change was ready and I wanted to merge this PR, I took the liberty of rebasing the history for a clean merge. I really hope you don't mind that I did it ✌🏽 Once again... thank you for your contribution. 😎

@imjaroiswebdev imjaroiswebdev merged commit 544d49e into PagerDuty:master Apr 5, 2023
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

Successfully merging this pull request may close these issues.

3 participants