docs: clarify missing wnd config parameter#41106
Conversation
mmattel
requested review from
DeepDiver1975,
jnweiger,
jvillafanez and
phil-davis
|
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
| * "user001@my.dom.com" will be mapped to the same windows user as | ||
| * "user001@not.mine.eu". | ||
| * | ||
| * - `CustomFile`: This mapping is reserved and should only be used on explicit request of ownCloud support. |
There was a problem hiding this comment.
I'd go further to say that it isn't implemented and there is no short term plan to add it.
Right now, it seems that there is a "magic button" to enable the mapping, but it isn't the case. It could take some time to implement the mapping, which might not be what the admin expects.
There was a problem hiding this comment.
The issue is as you described in the referenced wnd comment, that you want to keep it as reminder. If we write not implemented, than we need to remove it completely as this does not makes sense. To get a diplomatic balance between the two positions, I decided to write this, which is a dont do this and if you do you are on your own if stuff breakes.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
|
@jnweiger please apply a bit more care next time before merging - there is absolutely no need in having 3 commits now on master for this change. This should have been squashed Look how ugly this looks like .... |
docs: clarify missing wnd config parameter
* Merge pull request #41104 from owncloud/update_config_apps_sample docs: add missing WND Kerberos settings to config.apps.sample * Merge pull request #41106 from owncloud/wnd_fix_for_config_apps_sample docs: clarify missing wnd config parameter --------- Co-authored-by: Martin <github@diemattels.at>
* Merge pull request #41104 from owncloud/update_config_apps_sample docs: add missing WND Kerberos settings to config.apps.sample * Merge pull request #41106 from owncloud/wnd_fix_for_config_apps_sample docs: clarify missing wnd config parameter --------- Co-authored-by: Martin <github@diemattels.at>
4 participants
One WND config parameter needs clarification - this was missing.