movied users from old project to new project anover repository, updated tests, cleared warns and errors in tests #1

Closed
SergeyV wants to merge 0 commits from move_users into master
Collaborator

Movied users from old project to new project anover repository, updated tests, cleared warns and errors in tests

Movied users from old project to new project anover repository, updated tests, cleared warns and errors in tests
SergeyV added 1 commit 2024-04-01 12:45:31 +00:00
vbuglov requested changes 2024-04-01 14:57:12 +00:00
vbuglov left a comment
Owner

Я написал общие примеры, поправь пожалуйста. Мог не все заметить так как реквест большой, потом еще раз посмотрю

Я написал общие примеры, поправь пожалуйста. Мог не все заметить так как реквест большой, потом еще раз посмотрю
@ -44,3 +52,3 @@
firstTitle="Офлайн"
secondTitle="Онлайн"
:isCheck="selectedService.is_online"
:isCheck="selectedService?.is_online"
Owner

Старайся использовать camelCase

Старайся использовать camelCase
@ -81,2 +93,3 @@
id="proxyIp"
name="proxy_ip"
:value="selectedService.proxy_ip"
:value="selectedService?.proxy_ip"
Owner

camelCase

camelCase
@ -61,3 +69,3 @@
watch: {
isSaveData: function (newVal) {
console.log('newVal', newVal)
// console.log('newVal', newVal)
Owner

удали ненужные строки

удали ненужные строки
@ -67,3 +75,3 @@
},
isOnline: function (newVal) {
console.log('newVal', newVal)
// console.log('newVal', newVal)
Owner

удали ненужные строки

удали ненужные строки
@ -0,0 +6,4 @@
const randomNumber = Math.floor(Math.random() * chars.length)
pass += chars.substring(randomNumber, randomNumber + 1)
}
// console.log('pass', pass)
Owner

Удали ненужные строки

Удали ненужные строки
@ -3,2 +3,2 @@
const path = import.meta.env.VITE_API_ADDR
const UsersService = new Users(path)
// const path = import.meta.env.VITE_API_ADDR
// const UsersService = new Users(path)
Owner

Лучше удали если не будешь использовать

Лучше удали если не будешь использовать
@ -17,0 +12,4 @@
async addUserToService(userId) {
const selectedService = await this.serviceOfServices.getSelectedService()
if (!selectedService) {
console.error('addUserToService: Service not found')
Owner

Используй логгер

Используй логгер
@ -17,0 +16,4 @@
return {}
}
if (!userId) {
console.error('addUserToService: User not selected')
Owner

Используй логгер

Используй логгер
@ -17,0 +25,4 @@
async deleteUserFromService(userId) {
const selectedService = await this.serviceOfServices.getSelectedService()
if (!selectedService) {
console.error('removeUserFromService: Service not found')
Owner

Используй логгер

Используй логгер
@ -17,0 +29,4 @@
return {}
}
if (!userId) {
console.error('removeUserFromService: User not selected')
Owner

Логгер

Логгер
@ -18,0 +38,4 @@
async saveEditedData() {
const selectedService = await this.serviceOfServices.getSelectedService()
if (!selectedService) {
console.error('saveEditedData: Service not found')
Owner

логгер

логгер
@ -18,0 +51,4 @@
is_online: selectedService.is_online,
}
const newService = await this.serviceOfServices.createNewService(data)
console.log('newService', newService)
Owner

почисти от ненужных логов

почисти от ненужных логов
@ -0,0 +72,4 @@
const wrapper = mount(EditServiceCard, {
// shallow: true,
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +98,4 @@
const wrapper = mount(EditServiceCard, {
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +119,4 @@
const wrapper = mount(EditServiceCard, {
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +144,4 @@
const wrapper = mount(EditServiceCard, {
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +77,4 @@
const wrapper = mount(ServiceCard, {
// shallow: true,
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +108,4 @@
const wrapper = mount(ServiceCard, {
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +133,4 @@
const wrapper = mount(ServiceCard, {
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +163,4 @@
const wrapper = mount(ServiceCard, {
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +189,4 @@
const wrapper = mount(ServiceCard, {
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +235,4 @@
const wrapper = mount(ServiceCard, {
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
@ -0,0 +268,4 @@
const wrapper = mount(ServiceCard, {
global: {
plugins: [store]
Owner

Старайся через global: {
$mock: ...
}

Старайся через global: { $mock: ... }
SergeyV added 1 commit 2024-04-02 09:46:44 +00:00
SergeyV closed this pull request 2024-04-02 11:40:45 +00:00

Pull request closed

Sign in to join this conversation.
No Reviewers
No Label
No Milestone
No project
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: vbuglov/frontend-apps#1
No description provided.