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

Confidential is showed on email #3606 #3607

Merged
merged 2 commits into from Jan 6, 2016
Merged

Confidential is showed on email #3606 #3607

merged 2 commits into from Jan 6, 2016

Conversation

@xillibit
Copy link
Member

@xillibit xillibit commented Dec 29, 2015

No description provided.

@xillibit xillibit added pull labels Dec 29, 2015
@xillibit xillibit force-pushed the xillibit:k4.0_bugs6 branch from 95a6391 to 4265fe6 Dec 30, 2015
@810
Copy link
Member

@810 810 commented Dec 30, 2015

Remove confidential on email

  • Blue eagle - failed
  • Crypsis - failed
1 similar comment
@rich20
Copy link
Member

@rich20 rich20 commented Jan 3, 2016

Remove confidential on email

  • Blue eagle - failed
  • Crypsis - failed
@xillibit
Copy link
Member Author

@xillibit xillibit commented Jan 5, 2016

Should be better now

@810
Copy link
Member

@810 810 commented Jan 5, 2016

No,

now admins can't see the confidential on the page. Also I still see the confidential on the normal users email

@xillibit
Copy link
Member Author

@xillibit xillibit commented Jan 5, 2016

For me it's work correctly, the confidential are right displayed on messages but not in emails

@rich20
Copy link
Member

@rich20 rich20 commented Jan 5, 2016

For me it's work correctly, the confidential are right displayed on messages but not in emails

For me it works also correctly on messages

@810
Copy link
Member

@810 810 commented Jan 5, 2016

ok, works now

@810 810 added this to the 4.0.8 milestone Jan 5, 2016
810 added a commit that referenced this pull request Jan 6, 2016
Confidential is showed on email #3606
@810 810 merged commit 2b20782 into Kunena:develop Jan 6, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants