Skip to content

fix/admin dashboard template#45

Merged
CarmenDou merged 5 commits into
mainfrom
feat/admin-dashboard-template
May 27, 2026
Merged

fix/admin dashboard template#45
CarmenDou merged 5 commits into
mainfrom
feat/admin-dashboard-template

Conversation

@CarmenDou
Copy link
Copy Markdown
Contributor

@CarmenDou CarmenDou commented May 26, 2026

Summary by cubic

Fixes due-date timezone shifts in Tasks, tightens workspace RLS, and lets the insforge SDK handle the OAuth callback exchange. Also hides the unfinished Apps nav item.

  • Bug Fixes
    • Store tasks.due_date as date (was timestamptz) and parse/render as local dates; update sorting and overdue checks.
    • Keep form values as YYYY-MM-DD; stop converting to ISO strings when saving.
    • Let insforge auto-exchange the OAuth code; remove PKCE verifier storage and manual exchange logic.
    • Workspace RLS: include owners in SELECT so INSERT ... RETURNING works; pin owner_id in UPDATE WITH CHECK to prevent policy bypass.
    • Remove “Apps” from the sidebar until wired up.

Written for commit d58439a. Summary will update on new commits. Review in cubic

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 8 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread admin-dashboard/migrations/db_init.sql
Copy link
Copy Markdown
Member

@jwfing jwfing left a comment

Choose a reason for hiding this comment

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

LGTM, approved.

@CarmenDou CarmenDou merged commit a5223e1 into main May 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants