Skip to content

fix: fix detail view rendering#6

Merged
ontehfritz merged 1 commit intomainfrom
fix-no-records
Feb 4, 2026
Merged

fix: fix detail view rendering#6
ontehfritz merged 1 commit intomainfrom
fix-no-records

Conversation

@ontehfritz
Copy link
Contributor

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes list view rendering by adding tenant readiness checks to loading conditions, ensuring proper UI feedback during tenant initialization for system administrators.

Changes:

  • Added isTenantReady from useTenant() composable to three list views
  • Updated loading conditions to show loading state when tenant is not ready
  • Prevents blank screens during tenant initialization phase

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/views/RoleListView.vue Added isTenantReady check to display loading state during tenant initialization
src/views/PermissionListView.vue Added isTenantReady check to display loading state during tenant initialization
src/views/AccountListView.vue Added isTenantReady check to display loading state during tenant initialization

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ontehfritz ontehfritz merged commit 66f8c32 into main Feb 4, 2026
10 checks passed
@ontehfritz ontehfritz deleted the fix-no-records branch February 4, 2026 02:18
github-actions bot pushed a commit that referenced this pull request Feb 4, 2026
## [1.0.5](v1.0.4...v1.0.5) (2026-02-04)

### Bug Fixes

* fix detail view rendering ([#6](#6)) ([66f8c32](66f8c32))
@github-actions
Copy link

github-actions bot commented Feb 4, 2026

🎉 This PR is included in version 1.0.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants