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.

Sign up for GitHub

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

Jump to bottom

更新代码 #9

Merged
merged 6 commits into from
Nov 10, 2020
Merged

更新代码 #9

merged 6 commits into from
Nov 10, 2020

Conversation

KeRan213539
Copy link
Owner

No description provided.

horizonzy and others added 6 commits November 9, 2020 11:17
@horizonzy
[ISSUE-#4166] Optimize notifyCenter code (#4167)
48d0c68 
* enhance notify center code quality

* reverse the judge logic
@horizonzy
[ISSUE #3907] when publishConfig, if type is absent, set default valu…
39a2f20 
…e 'text' (#3915)

* when publishConfig, if type is absent, set default value 'text'

* 1.add publishConfig api(add param type)
2.check type is valid in nacos server
@KeRan213539
console\src\main\resources\static\console-fe 挪到根目录并改名为 console-ui (#4174
f01aae7 
)
@corey89757
[ISSUE #3102] Create utils class to nacos-common (#4165)
cf92d19 
* [ISSUE #3102] Create utils class to nacos-common.

Change-Id: I30970371b86a032366bd66c7b031a4cc3085512e

* [ISSUE #3102] Create utils class to nacos-common, fix check style errors.

Change-Id: I4711ad2b64a4265379f5d7df7434352e69c3760d
@KomachiSion
For #3607, #4078, #4170, Datasource pool configuration can be configu…
6b7fb05 
…red. (#4178)

* For #3607, #4078, #4170, Datasource pool configuration.

* Fix ci error.
@horizonzy
[ISSUE-#4179] Add index for db.user and db.password (#4186)
f808ffd 
* add index for db.user and db.password, to avoid some situation that it contains symbol ','

* add multi data source template

* add line to separate

* Revert "add line to separate"

This reverts commit 390eb6f

* Revert "add multi data source template"

This reverts commit 4470e63
@KeRan213539 KeRan213539 merged commit 1907df7 into KeRan213539:develop Nov 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@KeRan213539 @horizonzy @corey89757 @KomachiSion