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

Added links for titles A-E #289

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Added links for titles A-E #289

wants to merge 3 commits into from

Conversation

maxleee
Copy link

@maxleee maxleee commented Oct 11, 2019

Added the Goodreads links for titles A - E. Looks like prettier changed some of the formatting 😖 Let me know if this is an issue and I can re-do without prettier.

Closes #270

@maxleee maxleee requested a review from brittanyrw as a code owner October 11, 2019 16:10
@auto-comment
Copy link

auto-comment bot commented Oct 11, 2019

Thank you for opening this pull request, your code will be reviewed soon!
Reminder, please follow the contribution guidelines outlined in the README. This Pull Request should be for an Issue that you have been assigned.
If any updates need to be made, we will leave a comment for you here outlining exactly what you need to change for your pull request to be accepted. 💖

@brittanyrw
Copy link
Owner

Hi @maxleee, yes could you please update it so that it is not using prettier? Thanks!

@maxleee
Copy link
Author

maxleee commented Oct 11, 2019

Hi @brittanyrw I re-added the links. I'm still a git beginner, let me know if I need to do something else to resubmit this pull request? Thank you!

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.

Add Goodreads link for items starting with A to E in the data.js file
3 participants