refactor(components): Update props in SharePanel and ContentWrap

ZenUml · Nov 7, 2023 · eb08cde · eb08cde
1 parent 7460251
commit eb08cde
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 8 deletions.
diff --git a/src/components/ContentWrap.jsx b/src/components/ContentWrap.jsx
@@ -1062,8 +1062,10 @@ export default class ContentWrap extends Component {
 									}
 									content={
 										<SharePanel
-											id={this.props.currentItem.id}
-											author={window.user ? window.user.displayName : 'author'}
+											title={
+												this.props.currentItem && this.props.currentItem.title
+											}
+											author={window.user && window.user.displayName}
 											image={this.state.image}
 										/>
 									}

diff --git a/src/components/SharePanel.jsx b/src/components/SharePanel.jsx
@@ -27,7 +27,7 @@ export class SharePanel extends Component {
 	};
 
 	render() {
-		const { image, author } = this.props;
+		const { image, author, title } = this.props;
 		const { link, isLoading } = this.state;
 
 		return (
@@ -45,7 +45,7 @@ export class SharePanel extends Component {
 						<h4 style="margin-bottom: 8px;">Preview</h4>
 						<div className="preview">
 							<PreviewCard
-								title="Test Sequence Diagram"
+								title={title}
 								author={author}
 								description="Click and check the latest diagram. Install our Confluence plugin for an enhanced expperience when viewing in Confluence."
 								image={image}
@@ -75,9 +75,14 @@ export class SharePanel extends Component {
 	}
 }
 
+SharePanel.defaultProps = {
+	title: '',
+	author: '',
+	image: '',
+};
+
 SharePanel.propTypes = {
-	id: PropTypes.string,
-	dsl: PropTypes.string,
-	author: PropTypes.string,
-	image: PropTypes.string,
+	title: PropTypes.string.isRequired,
+	author: PropTypes.string.isRequired,
+	image: PropTypes.string.isRequired,
 };