-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor: refactoring links, popups, etc [WTEL-4531, WTEL-4528] #676
base: master
Are you sure you want to change the base?
Conversation
:file="csvFile" | ||
:shown="isUploadPopup" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Анімація працює, але файл не завантажує.
Навіть коли попап відсутній запускається created в uploadCSVMixin.js
wt-popup всередині використовує v-show. То получається що компонент постійно в DOM тому created викликається одразу.
@@ -56,6 +57,8 @@ const createVueInstance = () => { | |||
app.component(component.name, component); | |||
}); | |||
|
|||
app.component('adm-item-link', AdmItemLink); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
В деяких компонентах ми всеодно імпоруємо AdmItemLink.
Тоді для чого потрібно його робити глобальним тут?
:route-name="RouteNames.USERS" | ||
> | ||
<wt-button>neww</wt-button> | ||
</adm-item-link> | ||
<wt-headline-nav :path="path" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Получається тут треба рефакторити wt-page-header?
Напевно можна hide-primary просто зробти тру і стилі підправити, але чи це буде вірно?
throw new Error('Override id param for a component'); | ||
}, | ||
new() { | ||
return this.$route.params.new; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ось тут я шось не поняв, вроді в роуті нема параметра new
.
Чи я щось пропускаю?
Малось на увазі return this.$route.params.id = 'new';
?
No description provided.