Skip to content

Add the user to the order status change note - #3338

Open
robindevitt wants to merge 2 commits into
gocodebox:devfrom
robindevitt:issue/3039
Open

Add the user to the order status change note#3338
robindevitt wants to merge 2 commits into
gocodebox:devfrom
robindevitt:issue/3039

Conversation

@robindevitt

@robindevitt robindevitt commented Aug 26, 2026

Copy link
Copy Markdown

Description

Add the user to the order status change note.

Fixes #3039

How has this been tested?

Manually and reviewed via the order edit screen.

Screenshots

Screenshot 2026-08-26 at 10 13 07

Types of changes

The function add_note has a second parameter added_by_user which by default is false. So this change is setting the added_by_user parameter to true so the user details are shown in the notes.

By default it renders the users display_name. Referenced the screenshot for these below notes.
Note 1: Currently the note author was set to show as LifterLMS
Note 2: Showing the status update made by me with my display name being shown.
Note 3: I changed my display name and then changed the order status, to confirm it shows my updated display name.

Checklist:

  • This PR requires and contains at least one changelog file.
  • My code has been tested.
  • My code passes all existing automated tests.
  • My code follows the LifterLMS Coding & Documentation Standards.

@robindevitt
robindevitt requested a review from brianhogg as a code owner August 26, 2026 08:21
@brianhogg brianhogg moved this to Awaiting Review in Development Aug 26, 2026
@robindevitt
robindevitt changed the base branch from trunk to dev August 26, 2026 08:21
@brianhogg brianhogg moved this from Awaiting Review to Review in Progress in Development Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Review in Progress

Development

Successfully merging this pull request may close these issues.

Add the user who makes an order or student ernollment change to the log

2 participants