Skip to content

Create blog section ui/ux#33

Open
AlbinaMidwestern wants to merge 3 commits into
masterfrom
ch21173/create-blog-section-ui-ux
Open

Create blog section ui/ux#33
AlbinaMidwestern wants to merge 3 commits into
masterfrom
ch21173/create-blog-section-ui-ux

Conversation

@AlbinaMidwestern

Copy link
Copy Markdown
Collaborator

No description provided.

@shortcut-integration

Copy link
Copy Markdown

This pull request has been linked to Clubhouse Story #21173: Create Blog section ui/ux.

@Jared-Dev Jared-Dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We have some data we need to throw in here and we need the actual blog view to be built out.

Comment on lines +44 to +46
{/*<ShapeBottmRight>*/}
{/* <img src={imgL1FeatureCurve} alt="" className="img-fluid" />*/}
{/*</ShapeBottmRight>*/}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's just delete this.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Preview the blog specific page will be on the landing page?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No, we'll need to add a page. So in the repo we should add src/pages/blog/[postId].js

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Good. Where I can find data for blogs?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We talked about it in Standup, but for now lets just use the static data from the design file. We are still waiting on the backend to have live data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants