Atlas-Assistance-Dogs / atlas-dogs

Repository for Atlas Dogs flows in Salesforce
1 stars 1 forks source link

"volunteer" files added as volunteer in beta 2 not seen as admin user #541

Closed atlasjen closed 1 month ago

atlasjen commented 1 month ago

This is in beta2. I changed to user Peony Ho. I went to contact Lydia Eaton. https://atlasassistancedogs--beta2.sandbox.lightning.force.com/lightning/r/Contact/003U9000009Au5ZIAS/view I added 2 files, one "N/A" one "Volunteer". image

I also added an emergency relationship and assigned a program assignment. All seems good.

I logged out as Peony and went to look at Lydia Eaton. I see the N/A file, but not the Volunteer classified file. image

I logged back in as Peony and the volunteer file is there.

deb761 commented 1 month ago

I had forgotten to add your new user to the All group. You should see it now.

atlasjen commented 1 month ago

yep i see it now :)

So i tried soomething a bit sneaky-- as the volunteer user

image

and now when i go to look for the record, i can't see it, so clearly the edit did happen: image

i logged in as me, Helen Mcneil is there, and indeed, Volunteer has been removed as a position image

trying to think of what we want. we want the volunteer user to be able to - add new volunteers, so they need permissions to add users and change position to add volunteer. We also want them to be able to change volunteer status. maybe we accept it as planned behavior? ideas?

deb761 commented 1 month ago

I don't see a way around this behavior, really. Now, if Peony had created the Contact, she should still see that contact even without the Volunteer position.

atlasjen commented 1 month ago

so should she be getting the error about deleting the position? If we accept it as allowed behavior, i would think we wouldn't want the error.. or is it a timing thing.. that she's allowed to delete it while it has "volunteer" there, but when the page goes to refresh after the delete action, it then errors out as she's now not allowed to see the page?

atlasjen commented 1 month ago

i think the timing case is exactly what is happening.. i tihnk we consider this a won't fix re the error when removing volunteer as a position.

AND i think with your lastest changes we just tested this is good.