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

[Backport] Allow admins delete poll answer documents #3231

Merged
merged 3 commits into from
Jan 24, 2019
Merged

[Backport] Allow admins delete poll answer documents #3231

merged 3 commits into from
Jan 24, 2019

Commits on Jan 24, 2019

  1. Allow admins delete poll answer documents

    @microweb10 @decabeza
    microweb10 authored and decabeza committed Jan 24, 2019
    Configuration menu
    Copy the full SHA
    8f112cf View commit details
    Browse the repository at this point in the history

  2. Add missing i18n

    @decabeza
    decabeza committed Jan 24, 2019
    Configuration menu
    Copy the full SHA
    b588cd3 View commit details
    Browse the repository at this point in the history

  3. Fix destroy document specs 

    We were linking to the document url itself, which does not have a route
associated and so the specs fails

With this commit we are using the correct path to the destroy action of
the DocumentsController.

We are also using the referrer instead of a params[:from] attribute, as
it avoids having to pass an extra parameter, making the code prettier
and it works the same way
    @voodoorai2000 @decabeza
    voodoorai2000 authored and decabeza committed Jan 24, 2019
    Configuration menu
    Copy the full SHA
    dacc2d5 View commit details
    Browse the repository at this point in the history