Skip to content

HW Submission Fletcher Bonds#8

Open
fbonds wants to merge 2 commits intofrontend-application-development-uw20:masterfrom
fbonds:master
Open

HW Submission Fletcher Bonds#8
fbonds wants to merge 2 commits intofrontend-application-development-uw20:masterfrom
fbonds:master

Conversation

@fbonds
Copy link

@fbonds fbonds commented Apr 21, 2020

Week 2 HW Submission

Please fill out the information below in order to complete your assignment. Feel free to update this comment later if necessary.

  • Comfort rating on this assignment: 3 (the CSS.. I can't make this look pretty for my life)
  • Completion rating on this assignment: complete

Fletcher Bonds added 2 commits April 21, 2020 12:09
… errors with the exception of 'Dev Tools Failed To Parse SourceMap' which Google says is DevTools and not me. The css is not good. I added and removed MaterialUI as I couldn't figure out how to us it. I need to brush up on css I guess. Thanks.
"eslint": "^6.8.0",
"react": "^16.13.1",
"react-dom": "^16.13.1",
"react-scripts": "3.4.1"
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't forget to save dependencies (like prop-types) when installing

name: PropTypes.string.isRequired,
image: PropTypes.string.isRequired,
member: PropTypes.bool.isRequired
})
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice job being specific with your proptypes 👍

<small>{this.props.author.name}</small><br></br>
<small>{this.getMaDate(this.props.date)} - {this.props.minutesToRead} mins to read</small>
<small><BsBookmark /></small>

Copy link
Contributor

Choose a reason for hiding this comment

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

Missing audio available/member preview

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