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

Enhance the user experience in the scenario of submitting duplicate keys #3429

Merged
merged 8 commits into from
Dec 12, 2020

Conversation

klboke
Copy link
Contributor

@klboke klboke commented Dec 11, 2020

What's the purpose of this PR

增强重复键冲突场景下的用户体验
Enhance the user experience in the scenario of submitting duplicate keys

这个pr作用于文本编辑模式。我遇到过很多次,复制配置然后修改配置,最后提交的时候因为键值冲突,导致我之前所有的配置修改都没了,这个pr调整了一个键值冲突时的策略,当发生键值冲突时,编辑窗口不关闭,同时,通过异常消息告诉用户,存在哪些重复的键值
This pr acts in text editing mode. I have encountered many times, copying the configuration and then modifying the configuration. At the end of the submission, all my previous configuration changes were lost due to the key-value conflict. This pr adjusted a key-value conflict strategy. When a key-value conflict occurs When the editing window is not closed, at the same time, an exception message is used to inform the user of the duplicate key values

目前的:
current
image
修改后:
after edited
image

@codecov-io
Copy link

codecov-io commented Dec 12, 2020

Codecov Report

Merging #3429 (6942564) into master (03bc4b4) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3429   +/-   ##
=========================================
  Coverage     51.38%   51.39%           
- Complexity     2312     2313    +1     
=========================================
  Files           441      441           
  Lines         13821    13823    +2     
  Branches       1407     1408    +1     
=========================================
+ Hits           7102     7104    +2     
  Misses         6231     6231           
  Partials        488      488           
Impacted Files Coverage Δ Complexity Δ
...portal/component/txtresolver/PropertyResolver.java 89.77% <100.00%> (+0.23%) 45.00 <0.00> (+1.00)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 03bc4b4...6942564. Read the comment docs.

Copy link
Member

@nobodyiam nobodyiam left a comment

Choose a reason for hiding this comment

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

LGTM

@nobodyiam nobodyiam merged commit e64e733 into apolloconfig:master Dec 12, 2020
@nobodyiam nobodyiam added this to the 1.8.0 milestone Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants