Skip to content

Add books list feature as example #63

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

Merged
merged 2 commits into from
Oct 25, 2018
Merged

Add books list feature as example #63

merged 2 commits into from
Oct 25, 2018

Conversation

vincentchalamon
Copy link
Contributor

Fix #62

This PR only implements a feature presenting a list of books (no delete/create/edit/show) as an example, using @api-platform/client-generator.

The main theme remains the same as the welcome page (cf. Theme.js as template).

@dunglas
Copy link
Member

dunglas commented Oct 22, 2018

The generator has been designed to be used with Bootstrap, maybe should we install it instead?

@vincentchalamon
Copy link
Contributor Author

I redesigned the default books list according to the welcome template, to illustrate how to use a Theme in a react app.

@vincentchalamon
Copy link
Contributor Author

If you want to update this theme using bootstrap, I think it should be in a separate PR

@dunglas
Copy link
Member

dunglas commented Oct 23, 2018

Actually, Bootstrap should already be available: https://github.com/api-platform/api-platform/blob/master/client/package.json#L6-L7

This has been designed for this usage from the ground.

@vincentchalamon
Copy link
Contributor Author

But revamping the demo theme using bootstrap should be out of this PR

@dunglas
Copy link
Member

dunglas commented Oct 23, 2018

There are no revamping necessary. The generated code already supports Bootstrap.

@vincentchalamon vincentchalamon merged commit 0fec8ae into api-platform:master Oct 25, 2018
@vincentchalamon vincentchalamon deleted the feature/books-list branch October 25, 2018 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants