Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

Warning message in Subject of analyst email should be French as well for french report #2226

Closed
lmcbhvu opened this issue Aug 19, 2020 · 5 comments · Fixed by #2277
Closed
Assignees
Labels
bug Something isn't working low priority

Comments

@lmcbhvu
Copy link
Contributor

lmcbhvu commented Aug 19, 2020

Summary

Warning message in Subject of analyst email should be French as well

Steps to reproduce

  1. Submit french report content some of harmful words or offensive picture

page4 (1)

@lmcbhvu lmcbhvu added bug Something isn't working needs prioritization We have to prioritize this issue labels Aug 19, 2020
@dengcn dengcn self-assigned this Aug 19, 2020
@dengcn
Copy link
Contributor

dengcn commented Aug 19, 2020

This bug is similar to another one I am working on, I will take this one to save team time

@ngosset ngosset added low priority and removed needs prioritization We have to prioritize this issue labels Sep 8, 2020
@ngosset
Copy link
Contributor

ngosset commented Sep 8, 2020

@dengcn Can you verify if this was fixed by your other PR? If it is we can close this Issue.

@dengcn
Copy link
Contributor

dengcn commented Sep 8, 2020 via email

@dengcn
Copy link
Contributor

dengcn commented Sep 8, 2020

Norm: Just a correction, the issue is slightly different from that PR in review. So I just label this as similar. we need to keep this PR, and I will handle this when refactoring is clean.

@dengcn
Copy link
Contributor

dengcn commented Sep 10, 2020

Hong, FYI there is another bug error in this scenario "WARNING: self harm words detected" in Email subject is also missing French, to avoid further similar PR in the same area, I will fix this as well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working low priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants