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

Only add the delete url if there is a user id is present #625

Merged
merged 1 commit into from Oct 10, 2018

Conversation

Projects
None yet
2 participants
@mikemand
Contributor

mikemand commented Oct 10, 2018

Fixes #623

Only add the delete url if there is a user id is present
Fixes #623

Signed-off-by: Micheal Mand <micheal@kmdwebdesigns.com>
@nWidart

This comment has been minimized.

Show comment
Hide comment
@nWidart

nWidart Oct 10, 2018

Member

That was fast, thanks! :D

Member

nWidart commented Oct 10, 2018

That was fast, thanks! :D

@mikemand

This comment has been minimized.

Show comment
Hide comment
@mikemand

mikemand Oct 10, 2018

Contributor

No problem. You got your comment in 30 seconds after I opened the PR, LOL.

Also, you had already fixed it on the User transformer, so... Copy-paste for the win.

Contributor

mikemand commented Oct 10, 2018

No problem. You got your comment in 30 seconds after I opened the PR, LOL.

Also, you had already fixed it on the User transformer, so... Copy-paste for the win.

@nWidart nWidart merged commit 230d623 into AsgardCms:master Oct 10, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@mikemand mikemand deleted the mikemand:hotfix/issue-623 branch Oct 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment