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

Adds missing tests #51

Merged
merged 2 commits into from
Jun 11, 2020
Merged

Adds missing tests #51

merged 2 commits into from
Jun 11, 2020

Conversation

farridav
Copy link
Owner

@farridav farridav commented Jun 11, 2020 •
edited
Loading

  • Fills out all skeleton tests
  • Uses beautiful soup for sifting through HTML
  • Updates permission strings to be correct
  • Changes model permissions to look explicitly at view permissions
  • handle strings in hide apps and hide models

@farridav farridav changed the title Farridav/add missing tests Adds missing tests Jun 11, 2020
@farridav farridav mentioned this pull request Jun 11, 2020
Closed
@farridav farridav requested a review from TysonRV June 11, 2020 15:30
@farridav farridav merged commit 44b2dc0 into master Jun 11, 2020
@farridav farridav deleted the farridav/add_missing_tests branch June 11, 2020 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TysonRV TysonRV Awaiting requested review from TysonRV

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

Successfully merging this pull request may close these issues.
1 participant
@farridav