Skip to content

[Fix-9025] [UI Next] Rectify this issue about failing to save a process when the user is ordinary user.#9063

Merged
caishunfeng merged 2 commits into
apache:devfrom
calvinjiang:f-ordinary-user-create-workflow
Mar 21, 2022
Merged

[Fix-9025] [UI Next] Rectify this issue about failing to save a process when the user is ordinary user.#9063
caishunfeng merged 2 commits into
apache:devfrom
calvinjiang:f-ordinary-user-create-workflow

Conversation

@calvinjiang
Copy link
Copy Markdown
Contributor

@calvinjiang calvinjiang commented Mar 21, 2022

Purpose of the pull request

This PR will close #9025 .
close #9026

Brief change log

Verify this pull request

This change added tests and can be verified as follows:

  • Manually verified the change by testing locally.

The test results are as follows:

If the user is admin and the project has any existing task group, the user can assign a task group to the task.
image

If the user is ordinary user and the project hasn't any existing task group, the user can not assign a task group to the task.
image

@songjianet songjianet added bug Something isn't working UI ui and front end related labels Mar 21, 2022
@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Copy Markdown
Contributor

@devosend devosend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@songjianet songjianet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@caishunfeng caishunfeng merged commit 9b16324 into apache:dev Mar 21, 2022
songjianet added a commit that referenced this pull request Mar 22, 2022
#9062)

* [Feature][UI Next][V1.0.0-Alpha] Adjust all table column widths in this project.

* [Fix][UI Next][V1.0.0-Alpha] Fix workflow delete handle (#9057)

* [Fix][UI Next][V1.0.0-Alpha] User Manage Queue display error (#9060)

* fix package bug (#9051)

* [Fix][UI Next][V1.0.0-Alpha] Fix the monitoring center does not support multiple worker. (#9061)

* [Fix][UI Next][V1.0.0-Alpha] Remove the unused Ref.

* [Fix-9025] [UI Next] Rectify this issue about failing to save a process when the user is ordinary user. (#9063)

* modify the back-edn api

* fix this issue

* [Fix][UI Next][V1.0.0-Alpha]Change the column-config file name to column-width-config and COLUMN_CONFIG to COLUMN_WIDTH_CONFIG.

* [Bug][UI Next] Fix style errors (#9067)

* [Feature][UI Next][V1.0.0-Alpha] Adjust all table column widths in this project.

* [Fix][UI Next][V1.0.0-Alpha] Remove the unused Ref.

* [Fix][UI Next][V1.0.0-Alpha]Change the column-config file name to column-width-config and COLUMN_CONFIG to COLUMN_WIDTH_CONFIG.

Co-authored-by: Devosend <devosend@gmail.com>
Co-authored-by: zixi0825 <sunzhaohe0825@gmail.com>
Co-authored-by: songjianet <1778651752@qq.com>
Co-authored-by: calvin <jianghuachinacom@163.com>
Co-authored-by: labbomb <739955946@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working UI ui and front end related

Projects

None yet

5 participants