Vote #80917
完了Include updated_on and passwd_changed_on columns in users API response
0%
説明
Users CSV has updated_on and passwd_changed_on column since r19740. I think it is inconsistent that the values are not included in API response.
journals
--------------------------------------------------------------------------------
I've attached a patch to add these fields to user API, written by "@Takenori TAKAKI":https://www.redmine.org/users/145727 and me. Tests for these changes will be posted later.
--------------------------------------------------------------------------------
I've attached a patch to ensure these fields are correctly added.
But I'm afraid that these assertions might be too detailed. For now @Redmine::ApiTest::UsersTest@ doesn't have tests for individual fields except for id and some optional fields.
I wonder if we should add assertions for other fields too, or just drop this test patch to keep test code simple.
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
Setting the target version to 4.2.0.
--------------------------------------------------------------------------------
Committed the patch. Thank you for your contribution.
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
related_issues
relates,Closed,33347,Include updated_on and passwd_changed_on columns when exporting users to CSV
relates,Closed,33787,Redmine::ApiTest::UsersTest fail if config.time_zone is set