Skip to content

fix: remove dark mode, await saveAgentStats - #5

Open
IRONICBo wants to merge 12 commits into
memovai:mainfrom
IRONICBo:fix/db-persistence
Open

fix: remove dark mode, await saveAgentStats#5
IRONICBo wants to merge 12 commits into
memovai:mainfrom
IRONICBo:fix/db-persistence

Conversation

@IRONICBo

@IRONICBo IRONICBo commented Apr 6, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove dark mode toggle (ThemeToggle component, theme-light CSS, page.tsx reference)
  • Fix saveAgentStats fire-and-forget → awaited write to prevent silent data loss

Notes

Main branch already includes upstream DB-first architecture (no in-memory Maps, atomic chip ops, optimistic lock via saveWithRetry). This PR adds the two remaining fixes on top.

@vercel

vercel Bot commented Apr 6, 2026

Copy link
Copy Markdown

@IRONICBo is attempting to deploy a commit to the memov Team on Vercel.

A member of the Team first needs to authorize it.

@vercel

vercel Bot commented Apr 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agentcasino Ready Ready Preview, Comment Apr 7, 2026 4:40am

Request Review

@IRONICBo
IRONICBo requested a review from crispyberry April 6, 2026 04:03
@IRONICBo IRONICBo self-assigned this Apr 6, 2026
@IRONICBo
IRONICBo force-pushed the fix/db-persistence branch from 46ac17e to 15ec694 Compare April 6, 2026 04:47
@crispyberry

Copy link
Copy Markdown
Contributor

Please do not remove Dark Mode; I have reverted it to the previous style, allowing users to switch between modes freely.

@IRONICBo

IRONICBo commented Apr 6, 2026

Copy link
Copy Markdown
Member Author

Please do not remove Dark Mode; I have reverted it to the previous style, allowing users to switch between modes freely.

OK, maybe switch theme is better, dark mode icon is confused.

@crispyberry

Copy link
Copy Markdown
Contributor

Please do not remove Dark Mode; I have reverted it to the previous style, allowing users to switch between modes freely.

OK, maybe switch theme is better, dark mode icon is confused.

now there is no button here

@IRONICBo

IRONICBo commented Apr 7, 2026

Copy link
Copy Markdown
Member Author
image

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