You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/05/24 08:25:38 UTC
[GitHub] [pulsar-manager] tuteng commented on a change in pull request #388: Multi bookie cluster Support
tuteng commented on a change in pull request #388:
URL: https://github.com/apache/pulsar-manager/pull/388#discussion_r637776208
##########
File path: front-end/src/views/management/environments/index.vue
##########
@@ -76,183 +90,190 @@
</template>
<script>
-import { putEnvironment, fetchEnvironments, deleteEnvironment, updateEnvironment } from '@/api/environments'
-import { setEnvironment } from '@/utils/environment'
-import store from '@/store'
+ import { putEnvironment, fetchEnvironments, deleteEnvironment, updateEnvironment } from '@/api/environments'
+ import { setEnvironment } from '@/utils/environment'
+ import store from '@/store'
-export default {
- name: 'EnvironmentInfo',
- data() {
- return {
- environmentList: [],
- environmentTableKey: 0,
- environmentListLoading: false,
- textMap: {
- create: this.$i18n.t('env.newEnvDialogCaption'),
- delete: this.$i18n.t('env.deleteEnvDialogCaption'),
- update: this.$i18n.t('env.updateEnvDialogCaption')
- },
- dialogFormVisible: false,
- dialogStatus: '',
- form: {
- environment: '',
- broker: ''
- },
- temp: {
- 'name': '',
- 'broker': ''
- },
- superUser: false,
- roles: [],
- rules: {
- environment: [{ required: true, message: this.$i18n.t('env.envNameIsRequired'), trigger: 'blur' }],
- broker: [{ required: true, message: this.$i18n.t('env.serviceUrlIsRequired'), trigger: 'blur' }]
- }
- }
- },
- created() {
- this.getEnvironments()
- this.roles = store.getters && store.getters.roles
- if (this.roles.includes('super')) {
- this.superUser = true
- } else {
- this.superUser = false
- }
- },
- methods: {
- getEnvironments() {
- fetchEnvironments().then(response => {
- if (!response.data) return
- this.environmentList = []
- for (var i = 0; i < response.data.data.length; i++) {
- this.environmentList.push({
- 'environment': response.data.data[i].name,
- 'broker': response.data.data[i].broker
- })
+ export default {
+ name: 'EnvironmentInfo',
+ data() {
+ return {
+ environmentList: [],
+ environmentTableKey: 0,
+ environmentListLoading: false,
+ textMap: {
+ create: this.$i18n.t('env.newEnvDialogCaption'),
+ delete: this.$i18n.t('env.deleteEnvDialogCaption'),
+ update: this.$i18n.t('env.updateEnvDialogCaption')
+ },
+ dialogFormVisible: false,
+ dialogStatus: '',
+ form: {
+ environment: '',
+ broker: '',
+ bookie: ''
+ },
+ temp: {
+ 'name': '',
+ 'broker': '',
+ 'bookie': ''
+ },
+ superUser: false,
+ roles: [],
+ rules: {
+ environment: [{ required: true, message: this.$i18n.t('env.envNameIsRequired'), trigger: 'blur' }],
+ broker: [{ required: true, message: this.$i18n.t('env.serviceUrlIsRequired'), trigger: 'blur' }],
+ bookie: [{ required: true, message: this.$i18n.t('env.bookieUrlIsRequired'), trigger: 'blur' }]
}
- })
- },
- handleCreateEnvironment() {
- this.form.environment = ''
- this.form.broker = ''
- this.dialogFormVisible = true
- this.dialogStatus = 'create'
- },
- handleDeleteEnvironment(row) {
- this.temp.name = row.environment
- this.temp.broker = row.broker
- this.dialogFormVisible = true
- this.dialogStatus = 'delete'
+ }
},
- handleUpdateEnvironment(row) {
- this.form.environment = row.environment
- this.form.broker = row.broker
- this.dialogFormVisible = true
- this.dialogStatus = 'update'
+ created() {
+ this.getEnvironments()
+ this.roles = store.getters && store.getters.roles
+ if (this.roles.includes('super')) {
+ this.superUser = true
+ } else {
+ this.superUser = false
+ }
},
- handleOptions() {
- this.$refs['form'].validate((valid) => {
- if (valid) {
- switch (this.dialogStatus) {
- case 'create':
- this.createEnvironment()
- break
- case 'delete':
- this.deleteEnvironment()
- break
- case 'update':
- this.updateEnvironment()
- break
+ methods: {
+ getEnvironments() {
+ fetchEnvironments().then(response => {
+ if (!response.data) return
+ this.environmentList = []
+ for (var i = 0; i < response.data.data.length; i++) {
+ this.environmentList.push({
+ 'environment': response.data.data[i].name,
+ 'broker': response.data.data[i].broker,
+ 'bookie': response.data.data[i].bookie
+ })
+ }
+ })
+ },
+ handleCreateEnvironment() {
+ this.form.environment = ''
+ this.form.broker = ''
+ this.dialogFormVisible = true
+ this.dialogStatus = 'create'
+ },
+ handleDeleteEnvironment(row) {
+ this.temp.name = row.environment
+ this.temp.broker = row.broker
+ this.dialogFormVisible = true
+ this.dialogStatus = 'delete'
+ },
+ handleUpdateEnvironment(row) {
+ this.form.environment = row.environment
+ this.form.broker = row.broker
+ this.form.bookie = row.bookie
+ this.dialogFormVisible = true
+ this.dialogStatus = 'update'
+ },
+ handleOptions() {
+ this.$refs['form'].validate((valid) => {
+ if (valid) {
+ switch (this.dialogStatus) {
+ case 'create':
+ this.createEnvironment()
+ break
+ case 'delete':
+ this.deleteEnvironment()
+ break
+ case 'update':
+ this.updateEnvironment()
+ break
+ }
}
+ })
+ },
+ createEnvironment() {
+ const data = {
+ 'name': this.form.environment,
+ 'broker': this.form.broker,
+ 'bookie': this.form.bookie
}
- })
- },
- createEnvironment() {
- const data = {
- 'name': this.form.environment,
- 'broker': this.form.broker
- }
- putEnvironment(data).then(response => {
- if (!response.data) return
- if (response.data.hasOwnProperty('error')) {
+ putEnvironment(data).then(response => {
+ if (!response.data) return
+ if (response.data.hasOwnProperty('error')) {
+ this.$notify({
+ title: 'error',
+ message: response.data.error,
+ type: 'error',
+ duration: 2000
+ })
+ return
+ }
this.$notify({
- title: 'error',
- message: response.data.error,
- type: 'error',
+ title: 'success',
+ message: this.$i18n.t('env.addEnvSuccessNotification'),
+ type: 'success',
duration: 2000
})
- return
- }
- this.$notify({
- title: 'success',
- message: this.$i18n.t('env.addEnvSuccessNotification'),
- type: 'success',
- duration: 2000
+ this.dialogFormVisible = false
+ this.getEnvironments()
})
- this.dialogFormVisible = false
- this.getEnvironments()
- })
- },
- deleteEnvironment() {
- const data = {
- 'name': this.temp.name,
- 'broker': this.temp.broker
- }
- deleteEnvironment(data).then(response => {
- if (!response.data) return
- if (response.data.hasOwnProperty('error')) {
+ },
+ deleteEnvironment() {
+ const data = {
+ 'name': this.temp.name,
+ 'broker': this.temp.broker
+ }
+ deleteEnvironment(data).then(response => {
+ if (!response.data) return
+ if (response.data.hasOwnProperty('error')) {
+ this.$notify({
+ title: 'error',
+ message: response.data.error,
+ type: 'error',
+ duration: 2000
+ })
+ return
+ }
this.$notify({
- title: 'error',
- message: response.data.error,
- type: 'error',
+ title: 'success',
+ message: this.$i18n.t('env.deleteEnvSuccessNotification'),
+ type: 'success',
duration: 2000
})
- return
- }
- this.$notify({
- title: 'success',
- message: this.$i18n.t('env.deleteEnvSuccessNotification'),
- type: 'success',
- duration: 2000
+ this.getEnvironments()
+ this.dialogFormVisible = false
})
- this.getEnvironments()
- this.dialogFormVisible = false
- })
- },
- updateEnvironment() {
- const data = {
- 'name': this.form.environment,
- 'broker': this.form.broker
- }
- updateEnvironment(data).then(response => {
- if (!response.data) return
- if (response.data.hasOwnProperty('error')) {
+ },
+ updateEnvironment() {
+ const data = {
+ 'name': this.form.environment,
+ 'broker': this.form.broker,
+ 'bookie': this.form.bookie
+ }
+ updateEnvironment(data).then(response => {
Review comment:
comment It is recommended to use English so that all contributors can read it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org