-
Notifications
You must be signed in to change notification settings - Fork 63
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
[3.6] Backport of - Allow multiple values for same key in mass import csv #6031
[3.6] Backport of - Allow multiple values for same key in mass import csv #6031
Conversation
628aa45
to
434d1c7
Compare
I have no idea why the selenium tests for the Calendar are failing in the CI, locally they seem to work. I have not been able to fix them and as far as i can see nothing has changed in the calendar page..
|
2c9428e
to
3d9759d
Compare
@solth @oliver-stoehr I still have problems getting the tests working. What is failing now is the
|
@BartChris I will look into it and get back to you! |
I tried to reproduce the error on my local machine, but was not able to trigger the exception reliably. |
284695f
to
ca78f07
Compare
This reverts commit 4745541.
ca78f07
to
603946b
Compare
The issue with the failing Newspaper generation tests stems from the new test introduced in the Import Service integration tests. This test alters the meta/10.xml file, hence I reset this file after the completion of all tests within this class. Although the alterations to the Calendar tests are not strictly imperative for this pull request, considering the two instances of failure in the Selenium tests in the CI, I believe it might be good to incorporate them as well. |
Backport of #5998