拉代码 #5
Merged
拉代码 #5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…onfigDetails page (#3988) * add detail previous config history api * modify the previous config history url * modify log description * 详情页面添加“历史对比”按钮，点击可打开对比面板 * 点击“历史对比”按钮，获取上一版本数据 * 在查看历史对比的时候 不显示revert按钮 * append doc info * Add unit tests for ISSUE#3855 * Add the previous updates to main.js and main.css Co-authored-by: 赵延 <1060026287@qq.com>
* feature. add update and delete operation on instance's metadata 90% * 1.add unit test 2.fix-locateInstance error * format code style * add @link on class * move metadata operate to InstanceController. * format code 50% * complete bacth operation of metadata * remove unnecessary test unit * modify the variable name * perfect log info * modify consumer's dto to match it's description * 1.rename class OperationContext -> InstanceOperationContext 2.move Map operation from NamingUtils to MapUtils * check datum is null * code refactor. * 1. code refactor 2. modify the batch operate http param. just support (1 services : n instance) * 1.set default clusterName if instance not set 2.fix client params not set instances bug * npe fix * 1.remove consistencyType *, just support ephemeral or persist. 2.remove moditfy property method in InstanceOperationContext and InstanceOperationInfo, it just defined by constructor. * add instance controller test * fix code too length * update the param name * add feature's version of start
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily:
[ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=trueto make sure basic checks pass. Runmvn clean install -DskipITsto make sure unit-test pass. Runmvn clean test-compile failsafe:integration-testto make sure integration-test pass.