Skip to content
This repository has been archived by the owner on Oct 6, 2019. It is now read-only.

DNN-29429: Fix SMTP Server Tab UI Issue 3.0.x #1124

Conversation

kkadir
Copy link

@kkadir kkadir commented Aug 22, 2019

Fixes #1090

Summary

The <div className="tooltipAdjustment border-bottom"> tags are moved after the smtpSettingsVisible flag. This assures that a horizontal-line is visible only if that block is visible.

Confirmation Video

@kkadir kkadir changed the title DNN-29429: Move horizontal-line div tags after permission checks. DNN-29429: Fix SMTP Server Tab UI Issue 3.0.x Aug 22, 2019
@valadas
Copy link
Contributor

valadas commented Aug 23, 2019

Thanks @kkadir , those changes look good to me

Copy link
Contributor

@bdukes bdukes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me; I believe we're going to hold on merging until 9.4.0 is out.

@valadas valadas added this to the 3.0.0 milestone Sep 10, 2019
@valadas valadas added the bug Something isn't working label Sep 10, 2019
@valadas valadas modified the milestones: 3.0.0, 9.4.1 Sep 10, 2019
@valadas valadas merged commit 0d96510 into dnnsoftware:release/3.0.x Sep 17, 2019
mitchelsellers pushed a commit that referenced this pull request Sep 26, 2019
* Update "About" information for DNN Platform to be more relevant and current (#1094)

* Update to include relevant information and links for DNN Platform

* Update local resource file to include new content

* Update URLs to be absolute instead of using protocol-relative URLs

* Moved UNSAFE_componentWillMount to componentDidMount

* Fixed an issue where float where preventing collapsible height calculation

* Prevents chrome autofill

* Updates Dnn dependencies

* DNN-31578 - Not listing deleted children in pages Parent dropdown (#1099)

* Build front end projects in parallel using lerna (#1101)

* Added lerna config

* Sets up lerna to use yarn workspaces

* Adds checksum for 9.4.0 Platform

* Bumps version to 9.4.1 for next release (#1154)

* Fix incorrect sites loaded (#1159)

* DNN-33050 - Trigger reloadPortalList event with portalId and cultureCode

* DNN-33050 - Invoke getPortalSettings with current selected portalId and
cultureCode instead of getPortals which always point to current running
portal
- This is to load the correct portal settings based on user selection
when view portal settings

* DNN-29429: Move horizontal-line div tags after permission checks. (#1124)

* Fix tab sorting before they are serialized (#1134)

* DNN-32445 [Scheduler] Js error on changing scheduler server name (#1136)

* In Admin Logs, allow <pre> tags to be styled (#1142)

Fixes #1140

* Unable to set membership settings "Use Email Address as Username" ON (#1143)

* Fix to display IP Filter settings correctly (#1146)

* DNN-9532 - Fix to display IP Filter settings correctly
- Code refactor and minor bugs listed below
- Update state with the value from json response to ensure IP Filter
object in UI is updated
- Move specificityOptions and typeOptions initialization to constructor
- Move all the validation of the form upon saving to reduce code complexity
- Replace toggle option of RuleSpecificity to use const String of
IP_RANGE and  SINGLE_IP to make the code simpler and more readable
- Move code to clear SubnetMask just before update to prevent SubnetMask
become blank when user switch in between IP_RANGE and SINGLE_IP option

* DNN-9532 - Remove unnecessary blank line

* Modules get duplicated when try to localize modules (#1162)

* DNN-31428 - Remove overflow property after closing PersonaBar (#1164)

- revert to original style

* () Characters that are not illegal are considered illegal characters (#1166)

* Bumps Dnn dependencis to 9.4.0 release packages (#1169)

* Fix for upgrade issues of Personabar in 9.4.0 (#1170)

* Fix issues with upgrade of personabar to version 9.4.0 where old controllers were still in the DB and old packages were still registered

* Update projects

* Added message after localization update finishes (#1172)

* Removed wrong rc0001 references in csproj
valadas added a commit that referenced this pull request Sep 29, 2019
* Update "About" information for DNN Platform to be more relevant and current (#1094)

* Update to include relevant information and links for DNN Platform

* Update local resource file to include new content

* Update URLs to be absolute instead of using protocol-relative URLs

* Moved UNSAFE_componentWillMount to componentDidMount

* Fixed an issue where float where preventing collapsible height calculation

* Prevents chrome autofill

* Updates Dnn dependencies

* DNN-31578 - Not listing deleted children in pages Parent dropdown (#1099)

* Build front end projects in parallel using lerna (#1101)

* Added lerna config

* Sets up lerna to use yarn workspaces

* Adds checksum for 9.4.0 Platform

* Bumps version to 9.4.1 for next release (#1154)

* Fix incorrect sites loaded (#1159)

* DNN-33050 - Trigger reloadPortalList event with portalId and cultureCode

* DNN-33050 - Invoke getPortalSettings with current selected portalId and
cultureCode instead of getPortals which always point to current running
portal
- This is to load the correct portal settings based on user selection
when view portal settings

* DNN-29429: Move horizontal-line div tags after permission checks. (#1124)

* Fix tab sorting before they are serialized (#1134)

* DNN-32445 [Scheduler] Js error on changing scheduler server name (#1136)

* In Admin Logs, allow <pre> tags to be styled (#1142)

Fixes #1140

* Unable to set membership settings "Use Email Address as Username" ON (#1143)

* Fix to display IP Filter settings correctly (#1146)

* DNN-9532 - Fix to display IP Filter settings correctly
- Code refactor and minor bugs listed below
- Update state with the value from json response to ensure IP Filter
object in UI is updated
- Move specificityOptions and typeOptions initialization to constructor
- Move all the validation of the form upon saving to reduce code complexity
- Replace toggle option of RuleSpecificity to use const String of
IP_RANGE and  SINGLE_IP to make the code simpler and more readable
- Move code to clear SubnetMask just before update to prevent SubnetMask
become blank when user switch in between IP_RANGE and SINGLE_IP option

* DNN-9532 - Remove unnecessary blank line

* Modules get duplicated when try to localize modules (#1162)

* DNN-31428 - Remove overflow property after closing PersonaBar (#1164)

- revert to original style

* () Characters that are not illegal are considered illegal characters (#1166)

* Bumps Dnn dependencis to 9.4.0 release packages (#1169)

* Fix for upgrade issues of Personabar in 9.4.0 (#1170)

* Fix issues with upgrade of personabar to version 9.4.0 where old controllers were still in the DB and old packages were still registered

* Update projects

* Added message after localization update finishes (#1172)

* DNN-31960 Expand/Collapse All Fails After Moving Dragging Away The Only Child Item (#1176)

* DNN-28054 Expand All Fails After Adding Multiple Pages (#1175)

* Store date in "round-trip" format (#1179)

* Add missing keys for data consent (#1180)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants