Skip to content

[Bugfix] Add updateTime field into Mapper#313

Merged
SteNicholas merged 3 commits intoapache:mainfrom
XianmingZhou00:modify-sql
Jun 7, 2024
Merged

[Bugfix] Add updateTime field into Mapper#313
SteNicholas merged 3 commits intoapache:mainfrom
XianmingZhou00:modify-sql

Conversation

@XianmingZhou00
Copy link
Copy Markdown
Contributor

@XianmingZhou00 XianmingZhou00 commented Jun 6, 2024

Issue: #312

Purpose

  1. Fix the issue where the update time field on the user management list page and role management list page is displayed as null.
  2. Fix the updateTime field being null when inserting data.

Tests

  • SysRoleControllerTest#testQueryRole
  • SysRoleControllerTest#getRoleList
  • UserControllerTest#getUser
  • UserControllerTest#testListUsers

API and Format

No.

Documentation

No.

@SteNicholas SteNicholas changed the title [BugFix] Add the updateTime field to SQL [Bugfix] Add the updateTime field to SQL Jun 7, 2024
@SteNicholas
Copy link
Copy Markdown
Member

@XianmingZhou00, thanks for the contribution. Could you add test cases for this changes? Meanwhile, please add detail of this bugfix in the description of this pull request.

@SteNicholas SteNicholas changed the title [Bugfix] Add the updateTime field to SQL [Bugfix] Add updateTime field to SQL Jun 7, 2024
@SteNicholas SteNicholas changed the title [Bugfix] Add updateTime field to SQL [Bugfix] Add updateTime field into Mapper Jun 7, 2024
@SteNicholas SteNicholas merged commit 7d702aa into apache:main Jun 7, 2024
hdygxsj pushed a commit to hdygxsj/paimon-webui that referenced this pull request Jun 11, 2024
@XianmingZhou00 XianmingZhou00 deleted the modify-sql branch June 21, 2024 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants