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

admin ui: angular skeleton #138

Merged
merged 1 commit into from
Nov 8, 2018
Merged

admin ui: angular skeleton #138

merged 1 commit into from
Nov 8, 2018

Conversation

jma
Copy link
Contributor

@jma jma commented Nov 8, 2018

  • NEW angular UI project
  • NEW admin views: circulation settings and library settings
  • FIX restore the missing pipenv check command during the tests
  • FIX remove useless debug flag for tests

Signed-off-by: Johnny Mariéthoz [email protected]
Co-Authored-by: Igor Milhit [email protected]

Copy link

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

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

Installation ok. Tests ok. I could use the rero-ils interface without blatant new issues.

@jma jma removed the request for review from BadrAly November 8, 2018 09:46
@BadrAly BadrAly self-requested a review November 8, 2018 09:55
@BadrAly
Copy link

BadrAly commented Nov 8, 2018

The side bar is shifted all the way up when you select Library settings or circulation settings.

Copy link

@BadrAly BadrAly left a comment

Choose a reason for hiding this comment

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

No errors during the tests and interface works.

Copy link
Contributor

@rerowep rerowep left a comment

Choose a reason for hiding this comment

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

git log: two times Signed-off-by
git status: what to do with rero_ils/static/js/rero_ils/admin
Could we have different icons for different menues ?

* NEW angular UI project
* NEW admin views: circulation settings and library settings
* FIX restore the missing `pipenv check` command during the tests
* FIX remove useless debug flag for tests

Co-Authored-by: Igor Milhit <[email protected]>
Signed-off-by: Johnny Mariéthoz <[email protected]>
@jma jma force-pushed the maj-admin-views branch from a58124d to c58b823 Compare November 8, 2018 10:45
@jma jma merged commit 2809cfc into rero:master Nov 8, 2018
@jma jma deleted the maj-admin-views branch November 28, 2018 06:02
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.

4 participants