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

implementation of permission mechanism, insight page, refactor #132

Merged
merged 2 commits into from
Jun 14, 2020

Conversation

Rupeshiya
Copy link
Member


name: Pull Request template
about: Describe the PR.
title: ''
labels: ''
assignees: ''


Problem

Github Issue Number: #131

  1. As of now, only the creators of a particular post/event/project was allowed to perform edit or delete operation, but the admins should also have the access to perform these operations irrespective of the creator.
  2. There is no backend for the insight page of the client-side to check the overview of the organisation or members or search any members.l_

Solution of problem

Implemented backend for the above-mentioned problems:
So this PR includes:

  1. Backend for fetching org overview
  2. Backend for fetching personal overview
  3. Get all members info
  4. Search member by their names (first name, last name or full name)
  5. Added permission check for posts, comments, organizations, User delete backend
  6. User logout backend
  7. Bugfix in image upload (Organisation part: DB modified)
  8. Covered test cases for newly written backend

Type of Change

[✓] Bug fix
[✓] New Feature
[ ] Development of UI/UX prototypes
[✓] Small refactor
[ ] Change in Documentation

Checklist

[✓] My code follows the same style as the codebase
[ ] My Code change requires a change in documentation
[ ] I have updated the Readme accordingly
[✓] I made PR against development branch
[✓] I have run the test cases locally and it's passing.
[✓] I have squashed my commits

@devesh-verma devesh-verma merged commit ad28cb7 into codeuino:development Jun 14, 2020
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.

2 participants