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

Use DokuWiki's showuseras setting instead of custom user namespace, fix #151 #154

Merged
merged 1 commit into from May 16, 2015

Conversation

michitux
Copy link
Member

@michitux michitux commented Feb 1, 2015

This removes the custom user namespace links and replaces the whole custom link generation by standard DokuWiki code. Note that previously the id of the user page was the full name while now it is the user id (if enabled in DokuWiki).

This is a change which probably requires action by all people who were using the option to show the user as my guess is that in most cases showuseras isn't configured to give the behavior that people expect in the footer of the include plugin. Furthermore user pages no longer use the full name of the user but instead the user name as id and this cannot be changed.

Therefore I present this change as pull request so you can offer your opinion on this change.

This removes the custom user namespace links and replaces the whole
custom link generation by standard DokuWiki code. Note that previously
the id of the user page was the full name while now it is the user id
(if enabled in DokuWiki).
@ats20de
Copy link

ats20de commented Feb 1, 2015

As I'm not familiar with GitHub it took me some time to learn how to get the files but now I got them: it's exactly how it's supposed to be:

screenshot

Only cosmetics: Perhaps there should be a space between the symbol for the user and the user name for there is a space between middot and symbol, too (but this isn't really important).

Thanks for your help and fixing!

michitux added a commit that referenced this pull request May 16, 2015
Use DokuWiki's `showuseras` setting instead of custom user namespace, fix #151
@michitux michitux merged commit f8b1cc5 into master May 16, 2015
@michitux michitux deleted the showuseras branch May 16, 2015 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants