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

feat: privacy sheet updates #1634

Merged
merged 11 commits into from
Jun 25, 2024
Merged

feat: privacy sheet updates #1634

merged 11 commits into from
Jun 25, 2024

Conversation

tm-ruxandra
Copy link
Contributor

Description

Replace old ad-hoc privacy info modal implementation with one built on the standard OverlaySheet component. Update sheet content to match specification in design documents. Do minor cleanup in related areas.

Checklist

  • Corresponding issue has been opened

Related Issues

#755

Verification steps

Press the info button in the following areas and verify that the sheet works as expected:

  • Create project form
  • Create site form
  • Project input screen
  • Site dashboard screen

Copy link
Contributor

@knipec knipec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, so much cleaner, love this!

@ssfrancis
Copy link
Member

ssfrancis commented Jun 25, 2024 via email

@tm-ruxandra tm-ruxandra force-pushed the feat/privacy-sheet-updates branch from bfe5bb8 to 7996c0b Compare June 25, 2024 17:27
@tm-ruxandra tm-ruxandra merged commit f8dac35 into main Jun 25, 2024
9 checks passed
@tm-ruxandra tm-ruxandra deleted the feat/privacy-sheet-updates branch June 25, 2024 17:41
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.

4 participants