google / privacy-sandbox-aggregation-service Goto Github PK

Steps:

• Sign up with an email, e.g. [email protected]
• Grant "viewer" permission with the admin account
• Account "[email protected]" can see the job list after refresh
• Delete "[email protected]" with the admin account
• Account "[email protected]" sees the "Waiting for approval" page after refresh

Expect: the admin account can re-approve "[email protected]" from the "pending users"

Got: "[email protected]" is missing from "pending users"

Whenever I finish authentication and authorization, I will come back to this issue

Whenever I sign up with [email protected], I get a permissions error which stops the user from moving to the Jobs page. However, whenever I refresh the page, I then have permission to view the page.

Got:

1. Search a job, e.g. "66662e23-a0cd-4856-866f-2f4abe1f34ca" -> The only job will be listed
2. Delete the job ID -> Still the previous searching result

Expected: The page should list all the jobs. Currently there's no way to get back to the full list of jobs after searching.

Got:
Subjob items with name:
<aggregator>-<level>

Expected:
<level>-<aggregator>, since higher level subjobs can only start when all lower level subjobs finish.

The README currently lacks a section to inform users on how to add enforcement for the different user management roles. This is very important for users who want to later on add features to their control plane

Got:
There is a delete button for each job.

Expected:
We should remove it as the control plane will not delete jobs.

Got:
On the "filter" panel, users can filter jobs with status, created time and last updated time.
Searching by job ID is a separate function.

Expect:
Searching by ID sounds like another type of filter especially when we support "searching multiple jobs". Consider moving the search function as part of filter, so the user knows what they need to use for search.

main.js, main.js.map, main.css, main.css.map are all generated files that need to be removed from the repo and added to .gitignore

Add new readme instructions to make .firebaserc from .firebaserc.sample and add .firebaserc to .gitignore

Currently, all admin accounts are created using [email protected]. However, we should allow the user to change that email.