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

Redact switch #23

Open
0x4007 opened this issue Sep 30, 2024 · 5 comments · May be fixed by #24
Open

Redact switch #23

0x4007 opened this issue Sep 30, 2024 · 5 comments · May be fixed by #24

Comments

@0x4007
Copy link
Member

0x4007 commented Sep 30, 2024

Config to toggle redacts.

Used for private repos only.

redactPrivateRepoComments: boolean

Default we capture everything.

Maybe we shouldn't redact comments in the near future. And instead have a switch for it later when/if partners ever ask. I feel like most people won't care and it will give us more data to work with for research purposes in the near future.

I guess it would be fine to have it as an option in the configuration, because as long as not everybody can access the DB the content of private repositories cannot be read by outsiders.

Originally posted by @gentlementlegen in #20 (comment)

Copy link
Contributor

ubiquity-os bot commented Oct 1, 2024

! Fetching all pull requests failed!

@sshivaditya2019
Copy link
Collaborator

I think this task is overpaid, this is a simple fix. @0x4007

@sshivaditya2019
Copy link
Collaborator

/start

Copy link
Contributor

ubiquity-os bot commented Oct 2, 2024

@sshivaditya2019 the deadline is at Wed, Oct 2, 2:06 PM UTC

Copy link

Note

The following contributors may be suitable for this task:

gentlementlegen

0% Match ubiquity-os-marketplace/text-conversation-rewards#135

@0x4007 0x4007 added good first issue Good for newcomers and removed good first issue Good for newcomers labels Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants