Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Added the feature of, only superuser can delete the members in the the team and also changed the sizes of images in the team list. #332

Open
wants to merge 7 commits into
base: development
Choose a base branch
from

Conversation

DheerajMalisetty
Copy link
Contributor

@Aniketh01
Copy link
Member

Please provide me with substantial description with screenshots on why these changes were made so that I can merge this request.

Other than that, I am happy to merge the request :)

@DheerajMalisetty
Copy link
Contributor Author

In the case of "Each team page gets its name on the top inplace of word 'Team'" this commit. I made the changes to the webpage as mentioned in the below image.
screenshot from 2018-07-20 20-56-32
The above image is an example of a team page which gets its name on the top.

@Aniketh01
Copy link
Member

Cool. Please resolve the merge conflict.

@DheerajMalisetty
Copy link
Contributor Author

In the case of "Added the feature of, only superuser can delete the members in the team." Only the superuser has the access to delete the member. Normal users cannot delete the members. The below image shows it.
screenshot from 2018-07-20 21-00-48
In the above image it is the user who is logged in into the website.

@DheerajMalisetty
Copy link
Contributor Author

In the case of "Changed the Title of the Members page to Members" The Title page is Events details before, but I changed to 'Members'.
screenshot from 2018-07-20 21-24-11

Copy link
Collaborator

@chirathr chirathr left a comment

Choose a reason for hiding this comment

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

Why are you hard coding the images? These should be uploaded by the team member.

I think a better option would be to give access to staff(not superuser) and the team member who created the team.

@Aniketh01
Copy link
Member

Any changes in this pull request? @DheerajMalisetty

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants