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
Compose sidebar navigation improvements #1457
Compose sidebar navigation improvements #1457
Conversation
const block = new compose.PageBlockRecordList({ | ||
blockIndex: 0, | ||
options: { | ||
moduleID: module.moduleID, |
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.
You dont need to reinit the whole block, it has a watcher on options, so just updating the moduleID should be enough
Also can you turn in into the regular watcher structure (the handler ()), also you need to check if module exists
client/web/compose/src/components/Namespaces/NamespaceSidebar.vue
Outdated
Show resolved
Hide resolved
client/web/compose/src/components/Namespaces/NamespaceSidebar.vue
Outdated
Show resolved
Hide resolved
created () { | ||
const { meta = { ui: {} } } = this.module || {} | ||
|
||
let fields = ((meta.ui || {}).admin || {}).fields || [] | ||
fields = fields.length ? fields : this.module.fields | ||
|
||
const { moduleID } = this.module |
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.
You shouldnt do it here, you do it 5 rows above
@@ -347,6 +350,9 @@ export default { | |||
}, | |||
|
|||
adminRoutes () { | |||
const routeName = this.$route.name | |||
const pageName = routeName.endsWith('edit') ? 'admin.modules.edit' : 'admin.modules.record.list' |
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.
Wont this also trigger for admin.modules.record.edit?
2f2c2ec
to
1d22858
Compare
1d22858
to
703b8a1
Compare
703b8a1
to
8d9b4c8
Compare
The following changes are implemented
TODO: Summary
Changes in the user interface:
TODO: Add screenshots, recordings or remove this section
Checklist when submitting a final (!draft) PR