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

Delete user #73

Merged
merged 10 commits into from
Sep 4, 2019
Merged

Delete user #73

merged 10 commits into from
Sep 4, 2019

Conversation

mvatanya
Copy link
Collaborator

@mvatanya mvatanya commented Sep 1, 2019

Added delete feature in each user view so admin can delete the user.

@mvatanya mvatanya added the WIP label Sep 1, 2019
@mvatanya mvatanya removed the WIP label Sep 3, 2019
@Nalipp Nalipp added the WIP label Sep 3, 2019
@mvatanya mvatanya removed the WIP label Sep 3, 2019
@Nalipp
Copy link
Contributor

Nalipp commented Sep 3, 2019

We are also going to need to be able to change the status of a question to resolved.

I created an issue for you so that you can come back to this. Allow admin to mark question as answered #81

@Nalipp
Copy link
Contributor

Nalipp commented Sep 3, 2019

Line 83 in AdminPanel.js needs to be broken into multiple lines.
Right now it is 179 characters should limit 60-80 max.

frontend/src/AdminPanel.js Outdated Show resolved Hide resolved
@elie
Copy link
Contributor

elie commented Sep 3, 2019

@mvatanya please see comments above and fix the description for what you've done in this PR - good work so far!

@Nalipp
Copy link
Contributor

Nalipp commented Sep 4, 2019

Thank you!

@Nalipp Nalipp merged commit 2cdd678 into master Sep 4, 2019
@misscoded misscoded deleted the delete-user branch October 14, 2019 16:47
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.

3 participants