Skip to content

Use JS to show and hide extra forms on bulk adding page #1817

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

Open
symroe wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Use JS to show and hide extra forms on bulk adding page #1817

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
de8e0bc
Use JS to show and hide extra forms on bulk adding page
symroe Mar 30, 2022
8249081
Fixes bulk add error
VirginiaDooley Apr 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ynr/apps/bulk_adding/static/bulk_adding/js/bulk.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ $(function(){
}

// Remove empty fieldsets
var always_show_extra = $("form#bulk_add_form")[0].dataset['winnerCount'] * 3;
var always_show_extra = $("form#bulk_add_form")[0].dataset.winnerCount * 3;
Copy link
Copy Markdown
Contributor

@michaeljcollinsuk michaeljcollinsuk May 4, 2022

Choose a reason for hiding this comment

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

I haven't had chance to check this out locally to test, but I dont think this would fix the 500 error I mentioned in an old comment that happens when a ballot does not have winner_count set? I think that will need a change in the view.

There was also another bug that I came across when testing originally, that from memory, was that if I filled in multiple candidates using the new "add a row" button, and then submitting the form, only the last candidate that I entered was being created/shown for review. But as mentioned it was a while ago I tested it locally so will need to go through again when I have chance!

$("#bulk_add_form .sopn_adding_table tbody").each(function(i) {
var tbody = $(this);
if (tbody_is_empty(tbody)) {
Expand Down