codeforboston / maple

MAPLE makes it easy for anyone to view and submit testimony to the Massachusetts Legislature about the bills that will shape our future.
https://mapletestimony.org
MIT License
38 stars 106 forks source link

Privacy Setting - Adjusting "Browse Testimony" #1506

Closed kiminkim724 closed 1 month ago

kiminkim724 commented 2 months ago

Summary

Closes #1431 On Browse Testimony:

Checklist

Screenshots

Jeff as a public user:

Screen Shot 2024-03-12 at 5 24 33 PM

Jeff as a private user:

Screen Shot 2024-03-12 at 5 25 47 PM

Known issues

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. Go to Browse Testimony and check if private users have their names filtered from the Author List
  2. Go to profile and change privacy status to see if the hyperlinks and names appear in the Author list and Testimony results.
vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
maple-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 27, 2024 0:34am
mvictor55 commented 1 month ago

hmm @kiminkim724 i'm not able to reproduce without a few bugs (maybe a result of the preview environment)

image

Maybe you need to repull from dev and incorporate changes?

mertbagt commented 1 month ago

can you comment on this PR what the Integration Test issues are so we can get that looked at?

kiminkim724 commented 1 month ago

can you comment on this PR what the Integration Test issues are so we can get that looked at?

It's the publishTestimony tests under tests/integration/testimony.test.ts.

kiminkim724 commented 1 month ago

hmm @kiminkim724 i'm not able to reproduce without a few bugs (maybe a result of the preview environment)

  • All names appear as black text (not hyperlinked, regardless of privacy setting). I was able to click into Boyu's profile (see screenshot below)

I think this is mainly because there are some back-end changes to the PR, that requires to be merged into the dev environment to correctly add the "private" field to the testimony, which is what's checked for a user to be hyperlinked or not.

  • The private users seem to have their names blanked (see screenshot below). When I click on their testimony - it gives me a 404.

This is, unfortunately, the result of what happened with my script overriding the values in the older testimonies, so the testimonies before I think it was the end of November don't have information in Firebase, so we're unable to access the detailed view of those testimonies.