Bug 1016 - Operation Log needs to be updated when admin modifies the password for device admin

Review Request #433 — Created Aug. 28, 2024 and submitted

apoorva.sn
AMP
amp_3_7
1016
pmurugaiyan, prajesh, shuinvy

Problem

When admin modifies password for Device admin, even though the password is updated operation log prints it as failed and also it prints unnecessary logs which are not relevant to device admin

Solution

Some of the fields do not have custom edit function or there is no generic update fuction for those fields.
The update is happening as part of UserAuthorization->update() through the DB query when _insert_or_update is called below.
So have created non_editable_fields list and excluded those fields as they dont have seperate update functions

Tested locally by updating mail and phone number and there are no unnecessary logs created.
Have attached screenshot for the same

These is duplicate log shown, but there is a seperate bug to track that: bug 993


pmurugaiyan
  1. Ship It!
  2. 
      
shuinvy
  1. Ship It!
  2. 
      
prajesh
  1. Ship It!
  2. 
      
apoorva.sn
Review request changed

Status: Closed (submitted)

Loading...