Thomas Casteleyn
Thomas Casteleyn
That's why I specifically ask if the class is in the allowed matrix for any of the profiles this user has.
I have the same here: #197 It would really help a lot if the GitHub PR did already the necessary CI checks, so you immediately see if your PR can...
> And for now running any PR automatically in our test infrastructure would be risky (security)... GitHub has options for that where you need to approve to run the CI...
I would deprecate the `Email::LoadConfig` method and replace this line: https://github.com/Combodo/iTop/blob/7c7386afc71a9191f5fdd66888fe3ec8248ceaf0/core/email.class.inc.php#L163 With either `self::$m_oConfig = MetaModel::GetConfig();` or even `$oConfig = MetaModel::GetConfig();`. It's even more remarkable that the constructor does use...
I would fix the merge conflicts first 😉
No, but with it being base64 encoded, the user could copy and download/convert the data easier. But I like your suggestion as well.. Hmm 🤔
@piRGoif @Molkobain Do you have any update? What route should we take? Show content, show meta (size), show both?
Hi @piRGoif I added the HTML escaping..
I have done the change, but I feel like we still need to show the first x bytes (encoded)
Yeah, what is the use case of this whole feature.. It gives the ability for the user to see if the data makes any sense or not. But if you...