Skip to content

SDP-2102 fix: update broken SDP recipient_registration_attempts query - #2539

Merged
ElliotFriend merged 1 commit into
stellar:mainfrom
hypekostas:fix/update-sdp-broken-recipient-registration-attempts-query
Jul 1, 2026
Merged

SDP-2102 fix: update broken SDP recipient_registration_attempts query#2539
ElliotFriend merged 1 commit into
stellar:mainfrom
hypekostas:fix/update-sdp-broken-recipient-registration-attempts-query

Conversation

@hypekostas

Copy link
Copy Markdown
Contributor

What

Update DB query listed under Receiver Not Receiving OTP During Registration to use attempt_ts column instead of created_at.

Why

  • The current ORDER BY created_at DESC refers to a column that doesn't exist under the receiver_registration_attempts table, therefore running this query returns Error in query (7): ERROR: column "created_at" does not exist.
  • Instead, attempt_ts is the correct column name containing the intended rows.

Copilot AI review requested due to automatic review settings June 29, 2026 22:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a broken troubleshooting SQL query in the Stellar Disbursement Platform admin guide by updating the ORDER BY column to match the actual schema for receiver_registration_attempts.

Changes:

  • Update the troubleshooting query to sort by attempt_ts instead of the non-existent created_at column.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ElliotFriend ElliotFriend left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks for the fix, @hypekostas !!

@ElliotFriend
ElliotFriend merged commit 7a2d59b into stellar:main Jul 1, 2026
5 checks passed
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