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

Fix access method deletion in admin #469

Merged

Conversation

veggiematts
Copy link
Contributor

Test plan:

  1. On master or development branch, try to delete an unused access method in admin. The access method cannot be deleted.
  2. Apply the PR
  3. Try to delete an unused access method in admin. The unused access method can be deleted. Check that a used access method cannot be deleted.

@veggiematts veggiematts added the bug This is a bug (not an enhancement) label Aug 22, 2018
@jajm
Copy link

jajm commented Oct 18, 2018

Hi,

I followed the test plan and the patch works as expected.

@veggiematts veggiematts merged commit 1fa4708 into coral-erm:development Oct 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is a bug (not an enhancement)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants