-
Notifications
You must be signed in to change notification settings - Fork 6
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
Documentation notice after import #267
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just one suggestion to keep it consistent with the proposed design.
return ( | ||
<div className="doc-notice"> | ||
{ url ? ( | ||
<a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can use the <ExternalLink
component from Gutenberg instead of <a
as it adds the icon before the link as Design team has asked:
https://developer.wordpress.org/block-editor/reference-guides/components/external-link/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @HardeepAsrani, I thought about that but the Icon is after https://wordpress.github.io/gutenberg/?path=/story/components-externallink--default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it, makes sense.
@cristian-ungureanu Tested and everything's fine here 👍 |
🎉 This PR is included in version 1.2.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
Added the documentation notice after the import. The documentation notice is only visible if the import process is successful and if the starter site has the doc_url property defined in this file: https://github.com/Codeinwp/demo-data-exporter/blob/master/front-end/class-dde-frontend.php
Test instructions
functions.php
fileNote: I still have to add the doc_url parameter inside the exporter, I'm waiting for an answer regarding the links.
Closes Codeinwp/neve#3956, #247.