Skip to content

Remove _id suffix from BatchEdit changed -to-ones in the Audit Log - #8561

Draft
g1rly-c0d3r wants to merge 1 commit into
mainfrom
issue-8555
Draft

g1rly-c0d3r wants to merge 1 commit into
mainfrom
issue-8555

Conversation

@g1rly-c0d3r

@g1rly-c0d3r g1rly-c0d3r commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #8555

The backend has two different names for to-one relationships, an attname for the ID, as in how the underlying database identifies relationships, and a name, which is used to link to the actual relationship record.

On a form, when the to-one relationship is changed, we send the name field to the audit log. Before, BatchEdit would send the attname to the audit log, which just adds the _id suffix to the name. Now, BatchEdit sends the name field, in line with how a change through the forms would be logged.

main:

1789754260

issue-8555 (this pr):
1789754305

Checklist

  • Self-review the PR after opening it to make sure the changes look good and
    self-explanatory (or properly documented)
  • Add relevant issue to release milestone
  • Add automated tests

Testing instructions

@coderabbitai

coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@g1rly-c0d3r g1rly-c0d3r self-assigned this Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 📋Back Log

Development

Successfully merging this pull request may close these issues.

Batch Edit changes are added to SpAuditLog table with _id suffix

1 participant