SparkDevNetwork / Rock

An open source CMS, Relationship Management System (RMS) and Church Management System (ChMS) all rolled into one.
http://www.rockrms.com
580 stars 353 forks source link

Obsidian Notes Block Shows "Pending Approval" Label for Note Types with Enabled Approvals Turned Off #6066

Closed briankalwat closed 2 days ago

briankalwat commented 2 weeks ago

Description

We're currently in the process of updating our Rock instance from 17.0.27 to 17.0.31. During our QA process, we noticed that the Obsidian Notes block on person profile pages is showing labels that state "Pending Approval" even though we have Enabled Approvals disabled for the corresponding note type(s) (General Note, Event Registration Note, etc).

It seems like this is the conditional the handles whether the label is rendered or not, and it's odd that it does not appear to take into account the note type "Enabled Approvals" setting.

image

Actual Behavior

Notes that are of a note type that has "Enabled Approvals" disabled currently display the "pending approval" label in the new obsidian Notes block on person profile pages.

Expected Behavior

Notes that are of a note type that has "Enabled Approvals" disabled should not display the "pending approval" label in the new obsidian Notes block on person profile pages.

Steps to Reproduce

See the personal note at the top of Ted Decker's profile here: https://rockrmslatest.com/person/4 Enable Approvals is set to false for the Personal Note note type.

Issue Confirmation

Rock Version

17.0.31

Client Culture Setting

en-US