UI config to toggle whether user email shows up in Explore Users #336

avelino · 2016-12-02T06:08:00Z

In some scenarios it's undesirable to show the user's email in the Explore page, even though currently gitea shows only for signed-in users.

bkcsoft · 2016-12-02T06:57:06Z

Do you have a link to the original issue? As I remember there been a lengthy discussion about this one 😄

In short:
Is there a usecase for when we only want it gone in Explore? Why not always? (as always as possible at least). If not, shorten it to UI.ShowEmails. Other places where it shows up can be removed in a separate PR if necessary.

bkcsoft · 2016-12-02T06:58:10Z

modules/setting/setting.go

@@ -601,6 +602,8 @@ please consider changing to GITEA_CUSTOM`)
 	ShowFooterVersion = Cfg.Section("other").Key("SHOW_FOOTER_VERSION").MustBool()
 	ShowFooterTemplateLoadTime = Cfg.Section("other").Key("SHOW_FOOTER_TEMPLATE_LOAD_TIME").MustBool()

+	UI.ShowUserEmailInExplore = Cfg.Section("ui").Key("SHOW_USER_EMAIL_IN_EXPLORE").MustBool()


MustBool should take a boolean IIRC, and should default to true in this case to honor the ini-file :)

Also this line isn't necessary, see https://github.com/go-gitea/gitea/pull/336/files#diff-60f5fca6d29eafbb30eb645a0cd4041fR579

Yes, this line is no need since UI will be map from ini file.

This still applies, please add .MustBool(true)

strk · 2016-12-02T10:23:53Z

This has more to do with privacy than security (related to labels).
Are we sure this has to be in 1.0.0 ?

bkcsoft · 2016-12-02T10:30:40Z

This has nothing to do with security so I'm pushing this to v1.1 and re-labeling :)

lunny · 2016-12-24T02:40:46Z

Please resolve the conflict and we could merge it.

tboerger · 2016-12-29T10:40:12Z

conf/app.ini

@@ -60,6 +60,8 @@ FEED_MAX_COMMIT_NUM = 5
 THEME_COLOR_META_TAG = `#ff5343`
 ; Max size of files to be displayed (defaults is 8MiB)
 MAX_DISPLAY_FILE_SIZE = 8388608
+; Whether show the user email in the Explore Users page
+SHOW_USER_EMAIL_IN_EXPLORE = true


I would prefer SHOW_USER_EMAIL and use this setting for multiple places.

* github.com:go-gitea/gitea: (110 commits) add default values for SSH settings (go-gitea#500) slight optimization for GetUserRepositories (go-gitea#498) commithgraph / timeline (go-gitea#428) Permissions bug fix for webhooks Implementation of Folder Jumping Replace Gogs with Gitea (go-gitea#520) API endpoints for organization teams (go-gitea#370) Optimization for user.GetRepositoryAccesses to reduce db query times (go-gitea#495) Remove redundant query in collaborator API (go-gitea#516) ignore static files statstics for linguist Fix default cookie name fix installation page ssh domain unavilable fix 500 when delete orgnization and resolved go-gitea#486 At the locales replaced 6 with MIN_PASSWORD_LENGTH Added Dutch translations to the landingpage Improve the way how branches are deleted Improve issue references in markdown (go-gitea#471) API Endpoints for collaborators (go-gitea#375) Git LFS support v2 (go-gitea#122) Provide button to delete merged pull request (go-gitea#441) ...

go-gitea@66a1c59#r94122732

avelino · 2016-12-29T13:09:59Z

@lunny fixed conflict

bkcsoft · 2016-12-29T20:15:10Z

modules/setting/setting.go

+		FeedMaxCommitNum       int
+		ThemeColorMetaTag      string
+		MaxDisplayFileSize     int64
+		ShowUserEmailInExplore bool


Should probably be ShowUserEmail, and changed accordingly in the rest of the patch :) (since this will be used for more than just Explore in the long run :)

Sorry @bkcsoft , fixed

Bwko · 2016-12-30T13:04:16Z

@avelino Drone fails

bkcsoft · 2016-12-30T15:31:08Z

modules/setting/setting.go

@@ -197,12 +197,13 @@ var (
 	ScriptType   = "bash"

 	// UI settings
-	UI = struct {
+	UI struct {


This is wrong and makes Drone fail :(

derSuessmann · 2016-12-30T18:01:30Z

I have the same issue and have opened the #537 after some discussion on Gitter.
I think the UI option should be named UI.HideEmailAddress.
The reason is as follows:

There are different reasons why the email address should be hidden. Some can be found in issue Option to hide users' email address #537. The Gogs project discussed it already in August. The proposed solution was an opt-in per user.
The solution proposed in this PR fits my issue, as I would prefer hiding the email addresses for all users.
The broader solution would be a per user opt-in to show the email address.
If the option is called UI.HideEmailAddress it will be compatible with a latter developed opt-in. In the sense of: if false the user may set his email address to be shown and if the administrator decides that the email address should be hidden for all users it will override the individual settings.

As discussed in the Gogs project this will have no impact on the email address in the git log, if the user commits in Gitea. This should also be addressed in both cases.

bkcsoft · 2016-12-30T19:47:38Z

LGTM

avelino · 2016-12-30T19:58:59Z

Thank you for patience in reviewing my implementation, sorry for the delay in correcting the problems.

lunny · 2016-12-31T02:12:32Z

But we can also see the email on user profile page?

if you are active ShowUserEmail ref go-gitea#336 (comment)

lunny · 2016-12-31T09:25:31Z

The commit should be pushed to avelino:config_show_user_email_in_explore not master

if you are active ShowUserEmail ref go-gitea#336 (comment)

Bwko · 2016-12-31T16:07:28Z

modules/setting/setting.go

@@ -820,6 +821,8 @@ please consider changing to GITEA_CUSTOM`)
 	ShowFooterVersion = Cfg.Section("other").Key("SHOW_FOOTER_VERSION").MustBool(true)
 	ShowFooterTemplateLoadTime = Cfg.Section("other").Key("SHOW_FOOTER_TEMPLATE_LOAD_TIME").MustBool(true)

+	UI.ShowUserEmail = Cfg.Section("ui").Key("SHOW_USER_EMAIL").MustBool()


Please replace MustBool() with MustBool(true)

bkcsoft · 2016-12-31T16:20:06Z

But we can also see the email on user profile page?

@lunny IMO this PR only touches Explore-view to that's fine. On the other hand it would be fairly easy to fix that as well :)

lunny · 2017-01-01T02:50:47Z

LGTM

lunny · 2017-01-01T02:51:58Z

@bkcsoft I think they want to hide their email not only a single page.

@tboerger

* Bug fix for edit-hook API endpoint * Fixes issue #283 Delete old temp local copy before we create a new temp local copy * In the wiki title replace tab with a space (#371) * Correction LDAP validation (#342) * Correction LDAP username validation As https://msdn.microsoft.com/en-us/library/aa366101(v=vs.85).aspx describe spaces should not be in start or at the end of username but they can be inside the username. So please check my solution for it. * Check for zero length passwords in LDAP module. According to https://tools.ietf.org/search/rfc4513#section-5.1.2 LDAP client should always check before bind whether a password is an empty value. There are at least one LDAP implementation which does not return error if you try to bind with DN set and empty password - AD. * Clearing the login/email spaces at the [start/end] * Autogenerate Version on build (#190) * Autogenerate Version On Build * Fixes * Changed Version to v0.9.0 * balls * I hate newlines * Don't remove .VERSION-file on `make clean` * v0.9.0 => 1.0.0 * damn new-lines... * Always rebuild templates/.VERSION * Delete .VERSION * Update Makefile * bug fixed branch name for pushupdate (#380) * Don't use custom PBKDF2 function (#382) * fixed vulnerabilities (#392) * Update example install url from try.gogs.io to try.gitea.io (#385) * Update example install url from try.gogs.io to try.gitea.io * Ask if issue can be reproduced on try.gitea.io * Link try.gitea.io to the README * Fixing multiple docker issues (#386) * Added stupid docker task to makefile * Dropped unknown option PrintLastLog from docker ssh config * OpenSSH should log to docker stdout * Set random pw for docker git user, otherwise it is locked * Stop using templates and public within docker * Disable coverage report for now (#395) Until aircover gets updated so that it works again I have disabled to coverage report upload for now. * fixed vulnerabilities on deleting release (#399) * Revert "Disable coverage report for now" (#400) * Update gitea/sdk vendor (#406) * Fix typo (#407) * Remove fixed FIXME (#408) * fixed vulnerabilities labels (#409) * add ZH readme (#405) * Add support for using "libravatar" as the GravatarSource Just to make it easier for administrator to configure libre avatar, as it is done for "duoshuo" and "gravatar" * serve video files using the HTML5 video tag (#418) * serve video files using the HTML5 video tag * lint fix: add comment to IsVideoFile * change test mail subject and body to 'Gitea Test Mail!' Signed-off-by: Michael de Wit <[email protected]> * Fix random string generator (#384) * Remove unused custom-alphabet feature of random string generator Fix random string generator Random string generator should return error if it fails to read random data via crypto/rand * Fixes variable (un)initialization mixed assign Update test GetRandomString * Change default Android theme color meta tag (#389) * Do not override the binary version name from drone (#436) Signed-off-by: Thomas Boerger <[email protected]> * Remove remaining Gogs reference on locales and cmd (#430) * Add Korean support (#437) * Add Korean support * replace Gogs to Gitea * Update locales (#440) * Update locales * replace Gogs to Gitea * Fix homepage docs link broken (#417) * fix homepage docs link broken * fix homepage docs links after docs updated * Fix string format verbs (#3637) * issue comment api fix (#449) * ListAllInRepo & Delete Issue-Comments * Moar data in issue-comments * Bug fixes and tests for modules/base (#442) Also address other TODOs * UpdateIssueUsersByMentions was calling database write operations while (#443) a transaction session was in progress. MailParticipants was failing silently because of the SQLITE_LOCKED error. Make sure failures in MailParticipants enter the log, and pass on the transaction context. issue: let caller pass in database context, and use it issue_comment: obtain database context to pass to UpdateIssueMentions issue_comment: log any error from call to MailParticipants issue_mail: pass on database context to UpdateIssueMentions * update code.gitea.io/git (#450) * init script gentoo (#447) * init script for gentoo (#3761) * replace Gogs to Gitea * remove override port number * remove port * New settings option for a custom SSH host (#3763) (#446) * New settings option for a custom SSH host (#3763) * let default ssh listen addr empty * fix bug #1122 log.smtp receiver configure error (#3602) (#451) * Check if file is a symlink with web editor (#3687) (#445) * Check if file is a symlink with web editor (#3687) * editor checks for symlinks * translate file_is_a_symlink message * credit translation author * fix error constant * fix tag webhook 404 error (#420) * Bindata is optional and over-writable on restart (#354) * Moved conf assets into options folder * Dropped old bindata * Started to integrate options bindata and accessors * Do not enforce a builtin app.ini * Replaced bindata calls with options * Dropped bindata task from makefile, it's the generate task now * Always embedd app.ini to provide sane config defaults * Use sane defaults for the configuration * Defined default value for SSH_KEYGEN_PATH * Dropped "NEVER EVER MODIFY THIS FILE" header from app.ini * Fixed new paths in latest test additions * Drop bindata with make clean task * Set more proper default values * resolved #394 (#396) * Change screenshoots to Gitea (#454) * Fix race condition in unit test (#456) * Add default values for settings (#455) * add default values for settings * more default values * more default settings and labels resource * mv locale to options * Integrated an initial changelog (#457) * Simplified README and added a release badge (#460) * Dropped the status from readme, it's anway always outdated * Added version badge * Cleaned the install instructions * Dropped paragraphs that should be documented on the docs * Simplified the content, refer to the website * Dropped the logo, lets show screenshots * Fixed TRANSLATORS link * update zh readme (#462) * Fix broken godoc link and add arch in English README (#463) * fix broken godoc link and add arch in English README * typo * Fixed drone tagging * Properly handle drone tags and release/* branches (#466) * Raised version to 1.1.0 (#468) * Enables mssql support (#383) * Enables mssql support Port of dlobs work in gogs. Enables options in index.js Enables MSSQL as a database option in go. Sets ID to 0 on initial migration. Required for MSSQL insert statements. Signed-off-by: Beau Trepp <[email protected]> * Vendors in denisenkom/go-mssqldb Includes golang.org/x/crypto/md4 as this is required by go-msssqldb Signed-off-by: Beau Trepp <[email protected]> * API Endpoint for watching (#191) * Bug fixes for Issues filters (#413) Correctly handle simultaneous assignee/poster filters, and conflicting assignee filters * Added minimum password length to app.ini (#223) * Added sorting to the labels & milestones page (#199) * Simplified MinPasswordLength check (#475) * Added sorting to organizations, repos & users page (#222) * Organization webhook API endpoints * Update sdk * Implement sendmail (#355) * Implemented sendmail. This piggybacks on existing configuration to keep the change simple * Changed privicy of new sendSMTP and sendSendmail functions * Fixed Lint errors * Seperated SMTP and sendmail into their own senders * Making new structs private as they should not be used externally now * Added sendmail setting to ini file * Minor code cleanup * Provide button to delete merged pull request (#441) * provide button to delete merged pull request * golint fix * Git LFS support v2 (#122) * Import github.com/git-lfs/lfs-test-server as lfs module base Imported commit is 3968aac269a77b73924649b9412ae03f7ccd3198 Removed: Dockerfile CONTRIBUTING.md mgmt* script/ vendor/ kvlogger.go .dockerignore .gitignore README.md * Remove config, add JWT support from github.com/mgit-at/lfs-test-server Imported commit f0cdcc5a01599c5a955dc1bbf683bb4acecdba83 * Add LFS settings * Add LFS meta object model * Add LFS routes and initialization * Import github.com/dgrijalva/jwt-go into vendor/ * Adapt LFS module: handlers, routing, meta store * Move LFS routes to /user/repo/info/lfs/* * Add request header checks to LFS BatchHandler / PostHandler * Implement LFS basic authentication * Rework JWT secret generation / load * Implement LFS SSH token authentication with JWT Specification: https://github.com/github/git-lfs/tree/master/docs/api * Integrate LFS settings into install process * Remove LFS objects when repository is deleted Only removes objects from content store when deleted repo is the only referencing repository * Make LFS module stateless Fixes bug where LFS would not work after installation without restarting Gitea * Change 500 'Internal Server Error' to 400 'Bad Request' * Change sql query to xorm call * Remove unneeded type from LFS module * Change internal imports to code.gitea.io/gitea/ * Add Gitea authors copyright * Change basic auth realm to "gitea-lfs" * Add unique indexes to LFS model * Use xorm count function in LFS check on repository delete * Return io.ReadCloser from content store and close after usage * Add LFS info to runWeb() * Export LFS content store base path * LFS file download from UI * Work around git-lfs client issue with unauthenticated requests Returning a dummy Authorization header for unauthenticated requests lets git-lfs client skip asking for auth credentials See: git-lfs/git-lfs#1088 * Fix unauthenticated UI downloads from public repositories * Authentication check order, Finish LFS file view logic * Ignore LFS hooks if installed for current OS user Fixes Gitea UI actions for repositories tracking LFS files. Checks for minimum needed git version by parsing the semantic version string. * Hide LFS metafile diff from commit view, marking as binary * Show LFS notice if file in commit view is tracked * Add notbefore/nbf JWT claim * Correct lint suggestions - comments for structs and functions - Add comments to LFS model - Function comment for GetRandomBytesAsBase64 - LFS server function comments and lint variable suggestion * Move secret generation code out of conditional Ensures no LFS code may run with an empty secret * Do not hand out JWT tokens if LFS server support is disabled * API Endpoints for collaborators (#375) * Improve issue references in markdown (#471) * Improve issue references in markdown. (#3436) * Fix build * Fix lint * Fix comment typo * Improve the way how branches are deleted Delete branch from HeadRepo instead of BaseRepo Prevent the deletion of a master branch Show a yes/no overlay when you press the delete branch button * Added Dutch translations to the landingpage * At the locales replaced 6 with MIN_PASSWORD_LENGTH * fix 500 when delete orgnization and resolved #486 * fix installation page ssh domain unavilable * Fix default cookie name * ignore static files statstics for linguist * Remove redundant query in collaborator API (#516) * Optimization for user.GetRepositoryAccesses to reduce db query times (#495) * optimization for user.GetRepositoryAccesses to reduce db query times * fix missing cache * API endpoints for organization teams (#370) * Replace Gogs with Gitea (#520) * Implementation of Folder Jumping * Permissions bug fix for webhooks * commithgraph / timeline (#428) * Add model and tests for graph * Add route and router for graph * Add assets for graph * Add template for graph * slight optimization for GetUserRepositories (#498) * add default values for SSH settings (#500) * Repo permission bug fixes (#513) * Cache users on list releases (#527) * Tab on user profile to show starred repos (#519) * Tab on user profile to show starred repos * Make golint happy and use transactions on StarRepo function * x -> sess * Use sess.Close() instead of sess.Rollback() * Add copyright * Fix lint * Speed up conflict checking in pull request creation (#276) * Speed up conflict checking in pull request creation In order to check conflicts of a PR, we set up a working tree by cloning the base branch, which is quite time-consuming when the repository is huge. Instead, this PR uses `git read-tree` and `git apply --check --cached` to check conflicts. For #258 * Use $GIT_INDEX_FILE instead of --index-output to avoid lockfile problem The lockfile gets renamed to the final destination after the operation finishes. But it must be located in the same filesystem, which prevents us from using /tmp. * Temporary file names should not prefixed with '-' * push + pull now works with reverse proxy + basic auth on apache 2.4 * Notification - Step 1 (#523) * Notification - Step 1 * Add copyright headers * Cache issue and repository on notification model * Fix URL link on CONTRIBUTING Signed-off-by: Thibault Meyer <[email protected]> * Notifications - Step 2 * fix gofmt error Signed-off-by: Bo-Yi Wu <[email protected]> * replcae go fmt with gofmt command. Signed-off-by: Bo-Yi Wu <[email protected]> * Updated Dutch translations * Support http service graceful restart (#416) * support http service graceful restart * fix dependencies * API endpoints for forks (#509) * Restrict creating organisations by user (#193) * restrict creating organizations based on right on user * revert bindata.go * reverse vendor lib * revert goimports change * set AllowCreateOrganization default value to true * revert locale * added default value for AllowCreateOrganization * fix typo in migration-comment * fix comment * add coments in migration * fix windows build broken by #416 * Unit tests for token and update models * resolved #485: when migrate empty wiki repo, then ignore (#541) * Fix wrong anchors for non-latin headings (#3981) (#512) Change Javascript regular expression to match non-latin characters The regex comes from here: http://stackoverflow.com/questions/150033/regular-expression-to-match-non-english-characters#comment22322603_150078 And this patch should fixed these two issues: #3919 #3843 * resolved #517: fix admin ui data row missing * Replace pull request with issue at issue_template (#547) * Apply dockerfile updates to raspberry dockerfile (#548) * UI config to toggle whether user email shows up in Explore Users (#336) * UI config to toggle whether user email shows up in Explore Users * Recommendation made by @tboerger go-gitea/gitea@66a1c59#r94122732 * fixed typo, rename ShowUserEmailInExplore to ShowUserEmail * Fixed typo merged conflict * Hide email in the user profile page if you are active ShowUserEmail ref go-gitea/gitea#336 (comment) * Please replace MustBool() with MustBool(true) * Makefile: on Windows, executable should have ".exe" (#550) * Release API endpoints * Provide description for the INSTALL_LOCK config option (#554) * fix config session missing data row & resolved #517 (#549) * Do not get count on /api routers * More fine graned colors * Showing index in front of issue title * change the default action when deleting a release to not delete tag * Update maintainer list for LGTM * Fix benchmarktests (#557) * build failed by deleting release by API (#562) * bug fixed for fork repos (#560) * Update xorm and dependencies vendor for feature to dump to other database (#565) * update xorm and dependencies vendor for feature to dump to other database * fix golint * fix bug #564 (#567) * Add pagination for notifications * fix: Remove call to set GOMAXPROCS (#577) * Rename .gogs to .gitea and comply with github template guidelines (#568) (#582) Signed-off-by: Manuel Kuhlmann <[email protected]> * Fix typos in models/ (#576) * Check primary email address fields on CreateUser (#556) * Check primary email address fields on CreateUser As this check wasn't available, uid=1 (and possibly guests too, if registration is open) is able to register new users with existing email addresses. This leads to numerous 500 errors. * Update user.go * Lower the email first. Then check * Fix SQL bug in models/access (#583) Previously got a 'relation repo_access does not exist' error in User_GetRepositoryAccesses * Fix diff split view coloring (#553) (#584) Signed-off-by: Manuel Kuhlmann <[email protected]> * Fix unified diff view styling (#585) Signed-off-by: Manuel Kuhlmann <[email protected]> * Add the note forked from gogs (#590) * add the note forked from gogs * make the sentence simple * Clone tags within drone for proper version generation (#591) * Added 1.0.1 to changelog (#594)

UI config to toggle whether user email shows up in Explore Users

66a1c59

tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 2, 2016

bkcsoft suggested changes Dec 2, 2016

View reviewed changes

lunny added this to the 1.0.0 milestone Dec 2, 2016

lunny added topic/security Something leaks user information or is otherwise vulnerable. Should be fixed! topic/ui Change the appearance of the Gitea UI labels Dec 2, 2016

bkcsoft modified the milestones: 1.1.0, 1.0.0 Dec 2, 2016

bkcsoft added type/feature Completely new functionality. Can only be merged if feature freeze is not active. and removed topic/security Something leaks user information or is otherwise vulnerable. Should be fixed! labels Dec 2, 2016

tboerger reviewed Dec 29, 2016

View reviewed changes

tboerger added the issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail label Dec 29, 2016

Avelino added 2 commits December 29, 2016 11:05

Recommendation made by @tboerger

a232688

go-gitea@66a1c59#r94122732

bkcsoft suggested changes Dec 29, 2016

View reviewed changes

fixed typo, rename ShowUserEmailInExplore to ShowUserEmail

c3b7afd

bkcsoft approved these changes Dec 30, 2016

View reviewed changes

bkcsoft suggested changes Dec 30, 2016

View reviewed changes

lunny mentioned this pull request Dec 30, 2016

Option to hide users' email address #537

Closed

Fixed typo merged conflict

4c17aee

tboerger added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Dec 30, 2016

bkcsoft approved these changes Dec 30, 2016

View reviewed changes

avelino pushed a commit to avelino/gitea that referenced this pull request Dec 31, 2016

Hide email in the user profile page

2bd1e16

if you are active ShowUserEmail ref go-gitea#336 (comment)

Hide email in the user profile page

aa3b5b1

if you are active ShowUserEmail ref go-gitea#336 (comment)

Bwko reviewed Dec 31, 2016

View reviewed changes

Please replace MustBool() with MustBool(true)

e00b963

tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jan 1, 2017

lunny merged commit 787fda5 into go-gitea:master Jan 1, 2017

avelino deleted the config_show_user_email_in_explore branch January 1, 2017 04:31

tboerger removed the issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail label Jan 24, 2017

go-gitea locked and limited conversation to collaborators Nov 23, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

UI config to toggle whether user email shows up in Explore Users #336

UI config to toggle whether user email shows up in Explore Users #336

avelino commented Dec 2, 2016

bkcsoft commented Dec 2, 2016

bkcsoft Dec 2, 2016

bkcsoft Dec 2, 2016

lunny Dec 24, 2016

tboerger Dec 29, 2016

strk commented Dec 2, 2016

bkcsoft commented Dec 2, 2016

lunny commented Dec 24, 2016

tboerger Dec 29, 2016

avelino commented Dec 29, 2016

bkcsoft Dec 29, 2016

avelino Dec 29, 2016

Bwko commented Dec 30, 2016

bkcsoft Dec 30, 2016

derSuessmann commented Dec 30, 2016 •

edited

Loading

bkcsoft commented Dec 30, 2016

avelino commented Dec 30, 2016

lunny commented Dec 31, 2016

lunny commented Dec 31, 2016

Bwko Dec 31, 2016

bkcsoft commented Dec 31, 2016

lunny commented Jan 1, 2017

lunny commented Jan 1, 2017

UI config to toggle whether user email shows up in Explore Users #336

UI config to toggle whether user email shows up in Explore Users #336

Conversation

avelino commented Dec 2, 2016

bkcsoft commented Dec 2, 2016

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

strk commented Dec 2, 2016

bkcsoft commented Dec 2, 2016

lunny commented Dec 24, 2016

Choose a reason for hiding this comment

avelino commented Dec 29, 2016

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Bwko commented Dec 30, 2016

Choose a reason for hiding this comment

derSuessmann commented Dec 30, 2016 • edited Loading

bkcsoft commented Dec 30, 2016

avelino commented Dec 30, 2016

lunny commented Dec 31, 2016

lunny commented Dec 31, 2016

Choose a reason for hiding this comment

bkcsoft commented Dec 31, 2016

lunny commented Jan 1, 2017

lunny commented Jan 1, 2017

derSuessmann commented Dec 30, 2016 •

edited

Loading