Skip to content

Update profile attribute specs#9241

Merged
kbighorse merged 16 commits intomainfrom
LG-10902-Profile-attributes-doc-should-be-updated
Sep 21, 2023
Merged

Update profile attribute specs#9241
kbighorse merged 16 commits intomainfrom
LG-10902-Profile-attributes-doc-should-be-updated

Conversation

@kbighorse
Copy link
Contributor

@kbighorse kbighorse commented Sep 19, 2023

[skip changelog]

🎫 Ticket

LG-10902: Profile attributes doc should be updated

🛠 Summary of changes

This PR contains some cleanup encountered in connection with updating our profile documentation.

📜 Testing Plan

Tests pass

@kbighorse kbighorse marked this pull request as ready for review September 21, 2023 16:50
Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

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

fraud_pending_reason is used correctly in the existing specs and should not be deleted.

… set

Co-Authored-By: Sonia Connolly <2381438+soniaconnolly@users.noreply.github.com>
Co-Authored-By: Sonia Connolly <2381438+soniaconnolly@users.noreply.github.com>
Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

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

Paired on updating this. LGTM.

@kbighorse kbighorse merged commit 25e3076 into main Sep 21, 2023
@kbighorse kbighorse deleted the LG-10902-Profile-attributes-doc-should-be-updated branch September 21, 2023 20:33
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.

2 participants