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

Made package more user friendly #12

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

Pawel-IT
Copy link
Contributor

Changes:

  • Updated README.md with setup instructions for this repository
  • Made the seeder match the test cases. Seeded data before didn't work at all with the setup PostPolicy which was very confusing
  • Added some notes to UI to explain what is happening
  • Added links to all the routes and login info to welcome page and demo home page
  • Fixed couple test cases to work with the new seeder and UI changes
  • Added posts to the seeder to better illustrate examples outside of just the test cases
  • Changed the PUBLISHED scope for posts to show unpublished posts for those with permission to
  • Updated my_roles page to show the logged in user's roles not the User with id=1 roles

Before it was showing the first user role no matter which user was logged in which was confusing.
Added various notes about the role you're logged in with and what permissions you have/don't have. Also added all the links to the various routes.
It now matches the text class exactly.
Will check for roles and permissions so that they actually match the seeder.
To demo how the permissions work for guest users.
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.

1 participant