Skip to content

Verify Email - Loading Button#2071

Merged
Mephistic merged 2 commits intocodeforboston:mainfrom
mertbagt:verify_email_button
Mar 18, 2026
Merged

Verify Email - Loading Button#2071
Mephistic merged 2 commits intocodeforboston:mainfrom
mertbagt:verify_email_button

Conversation

@mertbagt
Copy link
Collaborator

@mertbagt mertbagt commented Mar 18, 2026

Summary

the verify email section was using the "info" style (as opposed to primary, secondary, warning, etc)

this was giving it a washed out look that could be mistaken for being disabled.

the styling was replaced with "secondary" on button, removed styling on text to give text a black color

Checklist

  • On the frontend, I've made my strings translate-able.
  • [n/a] If I've added shared components, I've added a storybook story.
  • [n/a] I've made pages responsive and look good on mobile.
  • [n/a] If I've added new Firestore queries, I've added any new required indexes to firestore.indexes.json (Please do not only create indexes through the Firebase Web UI, even though the error messages may reccommend it - indexes created this way may be obliterated by subsequent deploys)

Screenshots

image

Known issues

none

Steps to test/reproduce

For each feature or bug fix, create a step by step list for how a reviewer can test it out. E.g.:

  1. view profile with unverified account
  2. view page

@vercel
Copy link

vercel bot commented Mar 18, 2026

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

Project Deployment Actions Updated (UTC)
maple-dev Ready Ready Preview, Comment Mar 18, 2026 0:38am

Request Review

@Mephistic Mephistic merged commit 86b0cc2 into codeforboston:main Mar 18, 2026
8 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