Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[improvement-#11879]Keyword Search Improvement #12951

Merged
merged 8 commits into from
Dec 1, 2022

Conversation

fuchanghai
Copy link
Member

Purpose of the pull request

add a shortcut Key

Brief change log

add a shortcut Key

Verify this pull request

This pull request is code cleanup without any test coverage.

@github-actions github-actions bot added the UI ui and front end related label Nov 21, 2022
@SbloodyS SbloodyS added improvement make more easy to user or prompt friendly 3.2.0 for 3.2.0 version priority:middle labels Nov 21, 2022
@SbloodyS SbloodyS added this to the 3.2.0 milestone Nov 21, 2022
@codecov-commenter
Copy link

codecov-commenter commented Nov 21, 2022

Codecov Report

Merging #12951 (308f68b) into dev (c916c60) will decrease coverage by 0.14%.
The diff coverage is 21.64%.

❗ Current head 308f68b differs from pull request most recent head ff046a5. Consider uploading reports for the commit ff046a5 to get more accurate results

@@             Coverage Diff              @@
##                dev   #12951      +/-   ##
============================================
- Coverage     39.44%   39.30%   -0.15%     
- Complexity     4244     4274      +30     
============================================
  Files          1052     1069      +17     
  Lines         39857    40135     +278     
  Branches       4586     4593       +7     
============================================
+ Hits          15723    15774      +51     
- Misses        22356    22581     +225     
- Partials       1778     1780       +2     
Impacted Files Coverage Δ
...che/dolphinscheduler/alert/AlertSenderService.java 45.39% <0.00%> (-0.61%) ⬇️
...phinscheduler/api/controller/LoggerController.java 6.66% <0.00%> (ø)
...heduler/api/controller/TaskInstanceController.java 50.00% <0.00%> (+4.54%) ⬆️
...workflowInstance/WorkflowInstanceQueryRequest.java 0.00% <0.00%> (ø)
...eduler/api/service/impl/DataSourceServiceImpl.java 50.75% <ø> (-1.02%) ⬇️
...cheduler/api/service/impl/ExecutorServiceImpl.java 43.40% <0.00%> (-0.34%) ⬇️
...cheduler/common/constants/DataSourceConstants.java 0.00% <ø> (ø)
...e/dolphinscheduler/dao/entity/ProcessInstance.java 0.00% <ø> (ø)
...plugin/datasource/trino/TrinoDataSourceClient.java 0.00% <0.00%> (ø)
...r/master/dispatch/host/LowerWeightHostManager.java 0.00% <0.00%> (ø)
... and 74 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@fuchanghai
Copy link
Member Author

@Amy0104 PTAL

import { defineComponent, withKeys, PropType } from 'vue'
import { NInput } from 'naive-ui'

const placeholder ='Please enter keyword'
Copy link
Member

Choose a reason for hiding this comment

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

It needs to handle the different locales.

@sonarcloud
Copy link

sonarcloud bot commented Nov 29, 2022

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

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Member

@Amy0104 Amy0104 left a comment

Choose a reason for hiding this comment

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

LGTM

@Amy0104 Amy0104 merged commit 9e91a2b into apache:dev Dec 1, 2022
shangeyao pushed a commit to shangeyao/dolphinscheduler that referenced this pull request Dec 3, 2022
* [improvement-apache#11879]Keyword Search Improvement

* [improvement] add a search component

* [Feature][UI] Revise the  `Search` component.

* [improvement] add search components

* [improvement] fix

* [improvement] remove unUsed

* [improvement] remove unUsed

* handle the different locales.

Co-authored-by: Jackie <‘[email protected]’>
Co-authored-by: Amy <[email protected]>
ly109974 added a commit to ly109974/dolphinscheduler that referenced this pull request Apr 26, 2024
ly109974 pushed a commit to ly109974/dolphinscheduler that referenced this pull request Apr 26, 2024
cherry-pick [improvement-apache#11879]Keyword Search Improvement (apache#12951)

See merge request DataTech/dolphinscheduler!7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.2.0 for 3.2.0 version improvement make more easy to user or prompt friendly priority:middle UI ui and front end related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants