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.

Sign up for GitHub

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

Jump to bottom

Admin related changes #49

Merged
merged 2 commits into from
Dec 17, 2020
Merged

Admin related changes #49

merged 2 commits into from
Dec 17, 2020

Conversation

priyakvs
Copy link
Contributor

@priyakvs priyakvs commented Dec 10, 2020 •
edited

  • Changed admin page header to include pending books.
  • Removed cart from header for admins
  • Removed 'Destroy' for adopted and pending books.
  • Changed styles of button and display of login information.
  • Removed 'Add to cart' button for admins.
  • On logout, now it redirects to login page instead of root page.
  • Added test cases for all the new features added.

Pending items:

  • Change edit form like how Christa needs it.
  • When Christa moves pending book back to available (because of incomplete payment), author information should be removed from adopter table which automatically unlinks the adopter from products table.
  • Change look and feel of index and edit page if required.
  • Add validations to new and edit forms to have dropdown for status or in-editable product and adopter fields.

@hortongn hortongn linked an issue Dec 17, 2020 that may be closed by this pull request
Create admin page #27
Closed
@hortongn hortongn self-requested a review December 17, 2020 22:07
@hortongn hortongn merged commit 4f218ac into qa Dec 17, 2020
@hortongn hortongn deleted the admin_page_27 branch December 17, 2020 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hortongn hortongn hortongn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create admin page
2 participants
@priyakvs @hortongn