Skip to content

Fixed an issue where setting Use Permissions would not apply#26

Merged
valadas merged 1 commit into
DNNCommunity:masterfrom
valadas:Issue4
Jul 14, 2018
Merged

Fixed an issue where setting Use Permissions would not apply#26
valadas merged 1 commit into
DNNCommunity:masterfrom
valadas:Issue4

Conversation

@valadas

@valadas valadas commented Jul 14, 2018

Copy link
Copy Markdown
Member

Description of PR...

Fixed an issue where setting Use Permissions to No would have no effect after it was set to Yes.

Changes made

  • Added a check to show links when the setting if false. Before this pull request, it would only work when null, so it would stop working after initially set to True.

PR Template Checklist

  • Fixes Bug
  • Feature solution
  • Other

Please mark which issue is solved

Closes #4

@valadas
valadas merged commit 7b35154 into DNNCommunity:master Jul 14, 2018
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.

1 participant