Skip to content

[Fix-8836][UI Next][V1.0.0-Alpha] Rectify this issue about creating a file or directory with a wrong path.#8880

Merged
songjianet merged 3 commits into
apache:devfrom
calvinjiang:f-file-manage
Mar 15, 2022
Merged

[Fix-8836][UI Next][V1.0.0-Alpha] Rectify this issue about creating a file or directory with a wrong path.#8880
songjianet merged 3 commits into
apache:devfrom
calvinjiang:f-file-manage

Conversation

@calvinjiang
Copy link
Copy Markdown
Contributor

Purpose of the pull request

This PR will close #8836 .

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:

image

image

labbomb
labbomb previously approved these changes Mar 14, 2022
Copy link
Copy Markdown
Member

@labbomb labbomb left a comment

Choose a reason for hiding this comment

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

LGTM

@songjianet songjianet added bug Something isn't working UI ui and front end related labels Mar 14, 2022
devosend
devosend previously approved these changes Mar 14, 2022
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

@calvinjiang calvinjiang dismissed stale reviews from devosend and labbomb via 937dcbc March 14, 2022 13:08
@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

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

0.0% 0.0% Coverage
0.0% 0.0% Duplication

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

@songjianet songjianet merged commit e4dcf3d into apache:dev Mar 15, 2022
songjianet added a commit that referenced this pull request Mar 15, 2022
…eating a file or directory with a wrong path. (#8880)"

This reverts commit e4dcf3d.
zhongjiajie pushed a commit that referenced this pull request Mar 15, 2022
…eating a file or directory with a wrong path. (#8880)" (#8888)

This reverts commit e4dcf3d.
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

Development

Successfully merging this pull request may close these issues.

[Bug][UI Next][V1.0.0-Alpha] create directory/file path wrong

4 participants