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

Logout route #71

Merged
merged 3 commits into from
Apr 4, 2020
Merged

Logout route #71

merged 3 commits into from
Apr 4, 2020

Conversation

AlaaSaadeddin
Copy link
Collaborator

relates #20

@AlaaSaadeddin AlaaSaadeddin added Awaiting Review This PR is ready and waiting for review Backend labels Apr 4, 2020
@AlaaSaadeddin AlaaSaadeddin self-assigned this Apr 4, 2020
MohammedAlghazali
MohammedAlghazali requested changes Apr 4, 2020
View reviewed changes
Copy link
Member

@MohammedAlghazali MohammedAlghazali left a comment •
edited
Loading

Choose a reason for hiding this comment

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

check logout.js

server/controllers/routes/admin/logout.js Show resolved Hide resolved
@ranasobeid95 ranasobeid95 merged commit 068be41 into master Apr 4, 2020
@ranasobeid95 ranasobeid95 deleted the 20-logout branch April 4, 2020 22:31
@ranasobeid95 ranasobeid95 added Done and removed Awaiting Review This PR is ready and waiting for review labels Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MohammedAlghazali MohammedAlghazali MohammedAlghazali approved these changes

@ranasobeid95 ranasobeid95 ranasobeid95 approved these changes

@Mu7ammadAbed Mu7ammadAbed Mu7ammadAbed approved these changes

@rehabas rehabas Awaiting requested review from rehabas

Assignees

@AlaaSaadeddin AlaaSaadeddin

Labels
Backend Done
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AlaaSaadeddin @Mu7ammadAbed @ranasobeid95 @MohammedAlghazali