Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix] pre-handoff bug fixes #92

Merged
merged 6 commits into from
Aug 2, 2024

Conversation

ronniebeggs
Copy link
Collaborator

What's new in this PR

Description

  • Removed deep linking comments from /AllCases screen.
  • Made sure users are properly deleted from the "users" table.
  • Removed the password from the userMetaData on the /SignUp/Password screen (Passwords should be associated to a users account using the finishAccountSignUp function).
  • Removed fullName field from public users table.
  • Prevent users from signing up with an existing email (There is currently no check on the /SignUp/Password screen to ensure that an inputted email hasn't already been used).
  • Fix long case status text from being cut off on AllCases screen.
  • Standardized Variable Names Across the updates flow.
    • Change variable names for updates so it's clear which fields will show up where on a notification.
    • Update push edge function with changed variable names.
    • Remove hardcoded supabase environment variable from the send_notification function.
  • Change account passwords and create password documentation for new devs.
  • Fixed retool issues.

CC: @ronniebeggs

@ronniebeggs ronniebeggs self-assigned this Aug 2, 2024
Copy link

linear bot commented Aug 2, 2024

@ronniebeggs ronniebeggs merged commit 8b4416b into main Aug 2, 2024
2 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.

1 participant