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

Fixes Page tags missing when export and import into new site #3530

Merged

Conversation

tingung
Copy link
Contributor

@tingung tingung commented Jan 29, 2020

Fixes #3529

Summary

Support page tags (Term) during export and import site

  • Convert TabInfo.Terms.name into comma separated string and save as ExportTab.tags when export pages
  • Convert ExportTab.Tags into Term and save as page terms during import
  • Minor fix in TermHelper::ToTabTerms to get vocabulary based on Name and ScopeId to get vocabulary for particular portal
  • Move TermHelper from Dnn.PersonaBar.Extensions library to DotNetNuke.Library to allow usage from Dnn.ExportImport.Library

Demo video - https://drive.google.com/file/d/1wISHN11d0NTzSH5rdCtYqFoNHL7jskaw/view

- Convert TabInfo.Terms.name into comma separated string and save as ExportTab.tags when export pages
- Convert ExportTab.Tags into Term and save as page terms during import
…uke.Library

- Allow both DnnExportImport library and Dnn.PersonaBar.Extensions to use TermHelper
@tingung tingung changed the title Fixing Page tags missing when export and import into new site Fixes Page tags missing when export and import into new site Jan 29, 2020
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.

Looks good to me

@valadas valadas merged commit 04c480e into dnnsoftware:develop Feb 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Page tags missing when export and import into new site
3 participants