Skip to content

Hide email/password change for bots without credentials - #5151

Merged
hlbmtc merged 3 commits into
mainfrom
feat/hide-password-email-change-for-bots-without-credentials
Aug 26, 2026
Merged

Hide email/password change for bots without credentials#5151
hlbmtc merged 3 commits into
mainfrom
feat/hide-password-email-change-for-bots-without-credentials

Conversation

@hlbmtc

@hlbmtc hlbmtc commented Aug 26, 2026

Copy link
Copy Markdown
Contributor
image

https://metaculus.slack.com/archives/C01Q9AQBVHB/p1787721089296889

Summary by CodeRabbit

  • Improvements
    • Account settings now provide a cleaner experience for users without API credentials.
    • Credential-management options, including password and email controls, are hidden when unavailable.
    • Unnecessary dividers are removed when credential controls are not shown.

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 3c7c663b-a00e-4671-834f-bb76cc08cb99

📥 Commits

Reviewing files that changed from the base of the PR and between c8cbc05 and c31d466.

📒 Files selected for processing (2)
  • front_end/src/app/(main)/accounts/settings/account/components/api_access.tsx
  • front_end/src/app/(main)/accounts/settings/account/page.tsx

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

Account settings now hide credential controls for credential-less API forecasting bots. ApiAccess accepts a divider option so the page can remove the divider when those controls are absent.

Changes

Account settings credentials

Layer / File(s) Summary
Optional API access divider
front_end/src/app/(main)/accounts/settings/account/components/api_access.tsx
ApiAccess accepts withDivider, which defaults to true, and renders the divider only when enabled.
Credential-aware settings integration
front_end/src/app/(main)/accounts/settings/account/page.tsx
The page detects credential-less API forecasting bots. It conditionally renders the no-password banner, email editor, and password changer. It passes the credential state to ApiAccess.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to c31d4

This is a localized account-settings UI change, and no actionable merge-blocking risk remains beyond normal checks and review.

Poem

A rabbit checks the settings pane

Credentials hide like drops of rain
The divider hops away
When bot paws lack keys today
API access stays neat again

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: hiding email and password controls for bots without credentials.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/hide-password-email-change-for-bots-without-credentials

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hlbmtc
hlbmtc deployed to testing_env August 26, 2026 12:17 — with GitHub Actions Active
@hlbmtc
hlbmtc deployed to testing_env August 26, 2026 12:17 — with GitHub Actions Active
@hlbmtc
hlbmtc requested a review from ncarazon August 26, 2026 12:18
@hlbmtc hlbmtc changed the title Hide email/passwrod change for bots without credentials Hide email/password change for bots without credentials Aug 26, 2026
@hlbmtc

hlbmtc commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@github-actions

github-actions Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Cleanup: Preview Environment Removed

The preview environment for this PR has been fully removed.

Resource Status
🌐 Preview App Deleted
🗄️ PostgreSQL Branch Deleted
⚡ Redis Database Deleted
🔧 GitHub Deployments Removed
📦 Docker Image Retained (auto-cleanup via GHCR policies)

Cleanup triggered by PR close at 2026-08-26T12:57:41Z

@hlbmtc
hlbmtc merged commit 70819be into main Aug 26, 2026
17 checks passed
@hlbmtc
hlbmtc deleted the feat/hide-password-email-change-for-bots-without-credentials branch August 26, 2026 12:57
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