Skip to content

Fixes an issue where the "Show my email" logic was reversed#38

Merged
valadas merged 1 commit into
DNNCommunity:developfrom
juvander:FixFor_36
Jan 16, 2020
Merged

Fixes an issue where the "Show my email" logic was reversed#38
valadas merged 1 commit into
DNNCommunity:developfrom
juvander:FixFor_36

Conversation

@juvander

Copy link
Copy Markdown
Member

Fix for issue #36

@barnsley7

Copy link
Copy Markdown

thanks for the reply and the fix.

so is the fix suggesting to add this code:

 if (objRepository.ShowEMail == 1) {
}

to this file: Repository.ascx.cs

my only such file is inside a install/temp folder

@juvander

Copy link
Copy Markdown
Member Author

There is the whole release with all the other pull requests found in here https://github.com/juvander/DNN.Repository/releases/tag/04.00.001

@barnsley7

Copy link
Copy Markdown

thanks, I shall try that.

@valadas
valadas changed the base branch from master to develop January 16, 2020 20:12
@valadas valadas changed the title Fix for issue #36 Fixes an issue where the "Show my email" logic was reversed Jan 16, 2020
@valadas valadas added the bug Something isn't working label Jan 16, 2020
@valadas valadas added this to the 4.1.0 milestone Jan 16, 2020
@valadas
valadas merged commit 9c23d71 into DNNCommunity:develop Jan 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants