codeday-labs-2021 / bookworms Goto Github PK

View Code? Open in Web Editor NEW

1.0 1.0 5.0 2.94 MB

:book: :bug: bookworms is a CodeDay Labs project to create a web app to share your ideas of a book with others.

Home Page: https://codeday-labs.github.io/bookworms/

HTML 5.90% Go 36.73% JavaScript 51.27% CSS 6.10%

golang javascript reactjs vercel

bookworms's People

Contributors

Stargazers

Watchers

Forkers

jeromewu auvx13 veritem dandde

bookworms's Issues

Reverse chronological order

Homepage feed is currently not in a reverse chronological order (FIFO rather than LIFO)

Develop review like and filter API

User story:

Develop book review creation view

Wireframe:

Feel free to make it more beautiful. 😛

If you don't want to write all the css by yourself, one way is to use component libraries like material-ui, bootstrap or semantic.

On the right hand side of the review list view, there is a category filter to filter for certain categories, as our categories is defined by the user at the moment, we need an API to get a list of user defined categories to allow user to filter based on available categories.

Bootstrap frontend code in `client` folder

Right now in client folder, only an index.html is there for testing. As our plan is to use react.js to develop our frontend, it is common practice to use create-react-app to bootstrap your project first.

If it is done correctly. a package.json file should be inside client folder (ex. client/package.json)

Design and develop book review creation API

Story link: https://github.com/codeday-labs/bookworms/projects/1#card-64509226

We are going to use REST API as our API standard in this project. Please do the follow tasks to design and develop APIs for book review creation:

Update your API design here: https://github.com/codeday-labs/bookworms/tree/main/server/docs
Update code inside server folder

Develop book review list view

Card link: https://github.com/codeday-labs/bookworms/projects/1#card-64509461

In this view, an user can view all the book reviews.

Defer after error handling

There was a mistake for the place of calling defer func.

Take this one as an example:

	DB, err := db.DB()

	defer func() {
		if err := DB.Client().Disconnect(db.Ctx); err != nil {
			panic(err)
		}
	}()

	if err != nil {
		return nil, err
	}

As DB becomes nil when there is an error, this will cause a panic (nil pointer dereference) in defer func and it is not an expected behavior, so the defer should go after error handling:

	DB, err := db.DB()

	if err != nil {
		return nil, err
	}

        defer func() {
		if err := DB.Client().Disconnect(db.Ctx); err != nil {
			panic(err)
		}
	}()