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.

Sign up for GitHub

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

Jump to bottom

Change name to Analyze Comment Toxicity with Perspective API #10

Merged
merged 3 commits into from
Jul 24, 2022
Merged

Change name to Analyze Comment Toxicity with Perspective API #10

merged 3 commits into from
Jul 24, 2022

Conversation

i14h
Copy link
Collaborator

@i14h i14h commented Jul 21, 2022

Feedback during user research sessions was the name doesn't convey the purpose

@i14h i14h requested review from dslucas and reemaka July 21, 2022 18:34
dslucas
dslucas reviewed Jul 21, 2022
View reviewed changes
extension.yaml Outdated Show resolved Hide resolved
@i14h @dslucas
update copy
8ad9fa9 
Co-authored-by: dslucas <[email protected]>
@i14h
Copy link
Collaborator Author

i14h commented Jul 21, 2022

@dslucas thanks for the suggestion. PTAL

dslucas
dslucas approved these changes Jul 22, 2022
View reviewed changes
Copy link
Collaborator

@dslucas dslucas left a comment

Choose a reason for hiding this comment

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

Thanks!

@i14h i14h merged commit d96117a into main Jul 24, 2022
@i14h i14h deleted the i14h-naming-improvement branch July 24, 2022 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dslucas dslucas dslucas approved these changes

@reemaka reemaka Awaiting requested review from reemaka

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@i14h @dslucas