-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fix/3098 small bugs in test #51
Conversation
…hBucketPermissions, then users to support ManagedBy info.
ec876c4
to
7f06bb9
Compare
…, removed duplicate line.
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 need to resolve a conflict
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.
Double check import ordering. While the groupings are correct, you also want to do a secondary sort based on the from string value.
import { Permissions } from '@/utils/constants'; | ||
import { Button } from '@/lib/primevue'; | ||
import { usePermissionStore, useUserStore } from '@/store'; |
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.
@/lib
@/store
@/utils
@@ -1,7 +1,12 @@ | |||
<script setup lang="ts"> | |||
import { storeToRefs } from 'pinia'; | |||
import { onBeforeMount, ref, Ref, watch } from 'vue'; |
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.
Ref is actually a type
import type { Ref } from 'vue';
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.
all working for me now
Description
Types of changes
Checklist
Further comments