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

S23 user privacy setting #937

Closed
wants to merge 51 commits into from
Closed

S23 user privacy setting #937

wants to merge 51 commits into from

Conversation

ArabellaJi
Copy link
Contributor

@ArabellaJi ArabellaJi commented Jun 27, 2023

Allow users to choose which phone numbers they would like to show in their profiles and some other privacy options.

Fixes gordon-cs/gordon-360-ui#1308
Fixes gordon-cs/gordon-360-ui#1897

UI: gordon-cs/gordon-360-ui#1931

@ArabellaJi
Copy link
Contributor Author

The [UserPrivacy_Settings], [UserPrivacy_Fields]. and [UserPrivacy_Visibility_Groups] tables, and the users' privacy data based on their previous privacy setting have been added to the production Database by @amos-cha. Thank you!

amos-cha
amos-cha previously approved these changes Jul 22, 2023
Copy link
Contributor

@amos-cha amos-cha left a comment

Choose a reason for hiding this comment

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

lgtm

@russtuck
Copy link
Member

There are build errors after the merge from develop. Please fix them.

Copy link
Member

@russtuck russtuck left a comment

Choose a reason for hiding this comment

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

One question that needs an answer, and a few smaller things that it's good to get at the same time if possible.

Gordon360/Controllers/ProfilesController.cs Outdated Show resolved Hide resolved
Gordon360/Controllers/ProfilesController.cs Show resolved Hide resolved
Gordon360/Static Classes/Names.cs Outdated Show resolved Hide resolved
</member>
</members>
</doc>
<?xml version="1.0"?>
Copy link
Member

Choose a reason for hiding this comment

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

I'm puzzled about why github is showing that every single line of this file has changed Many seem the same, but some are different. I can't find a commit that changes more than a small piece, and blame shows most lines haven't changed for weeks or months. Do you know why it thinks every single line of this file changed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah this also confuses me. I don't know why GitHub thinks every line of code in this file has been changed.. Some new code was added, such as mail location and user privacy, but most of the code has not been changed.

russtuck
russtuck previously approved these changes Aug 25, 2023
Copy link
Member

@russtuck russtuck left a comment

Choose a reason for hiding this comment

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

Thank you for doing this! It's looking good, and I'm eager to see it go live.

@ArabellaJi
Copy link
Contributor Author

This PR is closed due to the difficulty of solving the merging conflicts here. This is the new PR: #1042.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 Points To measure progress Feature A new feature Profile Page s23 s24 Summer Practicum 2024
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add more granular Privacy Options for Fac/Staff Profile Show mobile phone # for facstaff
3 participants