Skip to content
This repository was archived by the owner on Nov 13, 2021. It is now read-only.

Scrub pinpoint status message#56

Merged
mitchellhenke merged 2 commits intomainfrom
mitchellhenke/scrub-pinpoint-status-message
Oct 29, 2021
Merged

Scrub pinpoint status message#56
mitchellhenke merged 2 commits intomainfrom
mitchellhenke/scrub-pinpoint-status-message

Conversation

@mitchellhenke
Copy link

No description provided.

message_id: message_response_result.message_id,
status_code: message_response_result.status_code,
status_message: message_response_result.status_message,
status_message: message_response_result.status_message.gsub(/\d/, 'x'),
Copy link
Author

Choose a reason for hiding this comment

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

\d is extremely unsophisticated, but works for now

Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

@mitchellhenke mitchellhenke merged commit 381131f into main Oct 29, 2021
@mitchellhenke mitchellhenke deleted the mitchellhenke/scrub-pinpoint-status-message branch October 29, 2021 16:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants