Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

[RFR] Use admin-on-rest #110

Merged
merged 14 commits into from
Sep 16, 2016
Merged

[RFR] Use admin-on-rest #110

merged 14 commits into from
Sep 16, 2016

Conversation

djhi
Copy link
Contributor

@djhi djhi commented Sep 4, 2016

  • Added basic admin configuration for products and orders
  • Added a custom rest client with jwt token
  • Added products in order creation and edition form

screen
screen

@fzaninotto
Copy link
Member

Very encouraging. It's so straightforward now!

@djhi
Copy link
Contributor Author

djhi commented Sep 7, 2016

Remaining problems:

I have to declare a route and related components for order-products. See marmelab/admin-on-rest#18

@djhi
Copy link
Contributor Author

djhi commented Sep 7, 2016

I'll see what I can do with a custom Layout as discussed

@djhi djhi changed the title [WIP] Use admin-on-rest [RFR] Use admin-on-rest Sep 7, 2016
Moved to last admin-on-rest
WIP: Trying to make it work
@djhi djhi changed the title [RFR] Use admin-on-rest [WIP] Use admin-on-rest Sep 8, 2016
@djhi djhi changed the title [WIP] Use admin-on-rest [RFR] Use admin-on-rest Sep 9, 2016
@@ -82,6 +82,11 @@ run-dev: ## Run all applications in development environment (using webpack-dev-s
@echo " Admin app: http://localhost:8080/admin"
@echo "Type 'make stop-dev' to stop the apps"

restart-dev: ## Stop all applications in development environment
Copy link
Member

Choose a reason for hiding this comment

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

s/Stop/Restart/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This sedy bot is awesome :) 👍 @Kmaschta

<div style={{ display: 'flex', flexDirection: 'column', minHeight: '100vh' }}>
<AppBar title={Title} iconElementRight={RightElement} />
<div className="body" style={{ display: 'flex', flex: '1', backgroundColor: '#edecec' }}>
<Menu />
Copy link
Member

Choose a reason for hiding this comment

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

You can put the <Menu> in the next line to make the content reads first in the source. The flex property insures it's displayed first.

@djhi djhi changed the title [RFR] Use admin-on-rest [WIP] Use admin-on-rest Sep 9, 2016
@djhi djhi changed the title [WIP] Use admin-on-rest [RFR] Use admin-on-rest Sep 9, 2016
@djhi
Copy link
Contributor Author

djhi commented Sep 9, 2016

Not happy yet with the layout & loading icon but it works

@@ -14,7 +14,7 @@ PM2_HOME ?= .pm2
help:
@grep -P '^[a-zA-Z_-]+:.*?## .*$$' $(MAKEFILE_LIST) | sort | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-30s\033[0m %s\n", $$1, $$2}'

# Initialization ===============================================================
#B Initialization ===============================================================
Copy link
Contributor

Choose a reason for hiding this comment

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

B?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants