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

Update "About" information for DNN Platform to be more relevant and current #1094

Merged
merged 4 commits into from
Aug 12, 2019

Conversation

david-poindexter
Copy link
Contributor

Resolves #81

Summary

See the referenced issue(s) for background information. Following are the implemented items:

  • An intro paragraph and logo was added for general "about" info for DNN Platform.
  • Version (Build) information is now retrieved dynamically and surfaced in the top right corner of the intro box.
  • Boxes with links added for DNN Docs, DNN Community and GitHub.
  • Title tags added to provide tooltips on hover.

Screenshot 2019-08-04 15 46 21

@david-poindexter david-poindexter added this to the 3.0.0 milestone Aug 8, 2019
@david-poindexter david-poindexter self-assigned this Aug 8, 2019
@david-poindexter david-poindexter changed the title Issue 81a Update "About" information for DNN Platform to be more relevant and current Aug 8, 2019
onEvoqClick() {
window.open("http://www.dnnsoftware.com/cms-features", "_blank");
onGitHubClick() {
window.open("//github.com/dnnsoftware/Dnn.Platform", "_blank");
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we add https to these links since they are external so we avoid a redirect from http to https ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, yes...after a bit of research, it appears using protocol-relative URLs is an anti-pattern. I will push a new commit with this change shortly. Thanks @valadas!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just pushed a new commit to address this concern.

</div>
</div>
</div>
);
}

UNSAFE_componentWillMount() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you move this into componentDidMount to avoid the soon to be deprecated method usage ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, great suggestion - thanks @valadas - standby for a new commit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, new commit pushed.

Copy link
Contributor

@valadas valadas left a comment

Choose a reason for hiding this comment

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

Awesome, looks good!

@valadas
Copy link
Contributor

valadas commented Aug 9, 2019

@ohine can we merge this now for the 9.4.0 next RC?

@david-poindexter
Copy link
Contributor Author

That would actually be great to merge for 9.4.0.

@mitchelsellers mitchelsellers merged commit 147748a into dnnsoftware:release/3.0.x Aug 12, 2019
@mitchelsellers
Copy link
Contributor

Merging this as we are still working on 9.4.0

@david-poindexter david-poindexter deleted the issue-81a branch August 13, 2019 19:31
mitchelsellers pushed a commit that referenced this pull request Sep 12, 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

* Reran yarn build
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants