Closed anmolsinghbhatia closed 4 days ago
The changes in this pull request focus on the ProjectViewsPage
component within the page.tsx
file. Two conditional checks have been reintroduced: one to verify the presence of workspaceSlug
and projectId
, returning an empty fragment if either is falsy; the other checks the currentProjectDetails?.issue_views_view
property, rendering an EmptyState
component if it is false. These modifications restore essential error handling and control flow for managing access permissions and project identification.
File Path | Change Summary |
---|---|
web/app/[workspaceSlug]/(projects)/projects/(detail)/[projectId]/views/(list)/page.tsx | Reintroduced conditional checks for workspaceSlug and projectId ; added rendering of EmptyState if issue_views_view is false. |
ProjectViewIssuesPage
component, relevant to the improvements in the ProjectViewsPage
.WorkspaceDraftHeader
component to improve validation and functionality, aligning with the focus on correct behavior in the ProjectViewsPage
.🐰 In the meadow, changes bloom,
With checks restored, there's less gloom.
If views are false, we won't despair,
An empty state shows we still care.
So hop along, let's celebrate,
Project views are now first-rate! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Changes:
This PR resolves an issue where clicking on “Project View” from a project with the view disabled resulted in an application error. Necessary updates have been made to fix this behavior.
Reference:
[WEB-2781]
Media:
Summary by CodeRabbit
EmptyState
component to inform users when project views are disabled, with a link to project settings.