qa.tech · PR testing – scene 5 (v05)

Every PR reviewed automatically → open the reviewed PR → tests pass → GitHub checks go green → ready to merge. Loops back to the list.

Pull Requests

Pull requests synced from GitHub, with QA.tech review results when available.

PostHog

AllOpenMergedClosed

FiltersOnly with reviews

Title

Repo

Author

Review

Created

Added middleware thingy

#41 · Merged

acme-signal

while

4 days ago

Add /network-test sandbox for custom request headers

#40 · Merged

acme-signal

while

4 days ago

Add HTTP Basic Auth protected route

#38 · Merged

acme-signal

lendai

Jul 1 2026

Add dark/light theme toggle

#37 · Open

acme-signal

lendai

Jun 22 2026

Add total and weighted value metrics to pipeline kanban headers

#30 · Open

acme-signal

while

Running…4

Mar 24 2026

Fix: Refresh data button on report detail page (DEA-27)

#29 · Open

acme-signal

while

Running…4

Mar 24 2026

Fix team invite confirmation and user list update

#28 · Open

acme-signal

while

Running…4

Mar 24 2026

Pull Requests

100

Add dark/light theme toggle #37 Open

lendaiQAdottech/acme-signalfeat/theme-togglemain

View on GitHub

ReviewFiles changedConversation

No regressions found

Jun 22 2026 · Auto-triggered when deployment was ready

Open conversationGitHub

Summary

All 7 tests pass across every scenario introduced by this PR. Activate dark mode via top bar and Switch back to light mode from dark confirm the toggle works bidirectionally with the correct icon swap and background change. Theme persists after page refresh verified the localStorage preference is saved and restored on reload. Dark mode consistent across app pages verified dark styling holds across Pipeline, Deals and Organizations after navigation.

Areas for improvement (2)

HTTP request failedUx

Failed to load Vercel preview feedback script. Third-party, unrelated to this PR.

HTTP request failedUx

Failed to load Vercel preview feedback script. Unrelated to PR changes.

Tests run (0)

Activate dark mode via top bar

45.4s

Switch back to light mode from dark

57.1s

Theme toggle on login page

45.2s

Theme persists after page refresh

64.8s

Dark mode consistent across app pages

117.5s

75.8s

148.6s

☰QAdottech / acme-signal ▾Type / to search+ ▾ ⊙ ▣ ▤ ◎

<> Code⚠ Issues 1 Pull requests 6▷ Actions▥ Projects☷ Wiki⛨ Security◉ Insights⚙ Settings

Add dark/light theme toggle #37

Ready to mergeCode ▾

Openlendai wants to merge 1 commit into main from feat/theme-toggle

Conversation 3Commits 1Checks 2Files changed 5+49 -5

lendai commented · on Jun 22Member

Summary

Adds a sun/moon theme toggle button to the top bar (authenticated app) and public header (login/signup pages)
Wires up the existing next-themes provider so dark mode styles already in the codebase actually work
Persists the user’s theme preference via localStorage, defaulting to system preference

Test plan

Click the sun icon — app switches back to light mode

Refresh the page — theme preference is preserved

Visit /login — theme toggle appears in the public header and works the same way

Verify sidebar, cards, and modals render correctly in both themes

Made with Cursor

👀 2

Add dark/light theme toggle to the top bar. de34d34

▲

vercel Bot commented · on Jun 22 · edited

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

ProjectDeploymentActionsUpdated (UTC)

acme-signal● ReadyPreview, CommentJun 22, 2026 12:31pm

⚙ Review with Vercel Agent

qa-tech-integration Bot commented · on Jun 22

👀 QA.tech will run a exploratory tests to review this PR as soon as a deployment is available for this PR. Alternatively, you can comment @qa.tech to manually trigger a review.

Learn more about configuring preview deployments.

What happens next

⏳ Waiting for your preview deployment or a comment mentioning @qa.tech
🔍 Running full end-to-end tests to validate your changes
📦 Delivering detailed results and actionable feedback

🤖 AI end-to-end testing powered by QA.tech

For organization ACME Corporation project ACME Signal CRM (Mqam). You can customize integration settings for this project.

💡 Comment @qa.tech to re-run tests, ask a follow-up, or request a focused review — e.g. @qa.tech test the checkout flow.

qa-tech-integration Bot deleted a comment from qa-tech-staging Bot · on Jun 22

▲vercel Bot deployed to Preview 2 months agoView deployment

qa-tech-staging Bot deleted a comment from qa-tech-integration Bot · on Jun 22

qa-tech-integration Bot deleted a comment from qa-tech-staging Bot · on Jun 22

qa-tech-staging Bot deleted a comment from qa-tech-integration Bot · on Jun 22

qa-tech-staging Bot commented · on Jun 22

💡 Tip

👍 Looks good

All 7 theme toggle tests passed across both the authenticated app and public pages.

Enable dark mode, Return to light mode, and Theme toggle icon state all confirmed the button works correctly in both directions with the icon updating as expected. Theme persists across navigation verified the ThemeProvider carries the selection through page transitions (pipeline → deals → organizations). Dark mode rendering confirmed layouts, cards, and tables stay readable across the pipeline, organizations list, and organization detail pages. The public header integration was tested on both Login page theme toggle and Signup page theme toggle — toggle visible in header, dark mode applies correctly on both.

Note: Theme persistence after a full page reload (localStorage rehydration) was exercised implicitly within each test session; a dedicated cross-session persistence test would give additional confidence.

▶ ✅ 8 passed tests

📊 View the full review on QA.tech →

💬 Leave feedback on this review → — your input teaches the agent to review this project better on future PRs.

🤖 AI end-to-end testing powered by QA.tech

For organization ACME Corporation project ACME Signal CRM (Mqam). You can customize integration settings for this project.

This branch was successfully deployed

1 active deployment

Show environments

All checks have passed

3 successful checks

No conflicts with base branch

Merging can be performed automatically.

Merge pull requestMerging…▾You can also merge this with the command line. View command line instructions.

Still in progress? Convert to draft

WritePreview

Add your comment here...

📝 Markdown is supported🖼 Paste, drop, or click to add files

⇄ Close pull requestComment

Reviewers

Suggestions

whileRequest

Still in progress? Convert to draft

Assignees

No one—assign yourself

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

None yet

Notifications

🔔 Subscribe

You’re not receiving notifications from this thread.

1 participant

🔒 Lock conversation

Speed 1.0×

Stage: PR list