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

ISSUE-368: edtf_2_iso_date #370

Merged
merged 4 commits into from
Oct 16, 2023
Merged

ISSUE-368: edtf_2_iso_date #370

merged 4 commits into from
Oct 16, 2023

Conversation

DiegoPino
Copy link
Member

What? see #368

edtf_2_iso_date which is a filter returns always an array. Either empty if we could not parse it, wrong data, etc. a single 'Y-d-m' value or two 'Y-d-m' if a range/uncertainty with 2 possible dates.

Still, good to test with our plugins/iiif tiles and clustering
edtf_2_iso_date returns always an array. [] always. ok?
@DiegoPino DiegoPino self-assigned this Sep 29, 2023
@DiegoPino DiegoPino added this to the 1.2.0 milestone Sep 29, 2023
@DiegoPino DiegoPino added metadata Meta(l) data Twig Extension Make those flowers reach the sun labels Sep 29, 2023
@DiegoPino
Copy link
Member Author

Note: this pull contains also #366. Just bc whoever gets merged first wins (and if that gets merged first this one will be 1:1/no need to rebase)

@alliomeria this here fixes the case where an legacy OSD viewer fails to detect (bc code was updated/settings were behind) the lack of IIIF driven manifests and triggers
Warning: Undefined variable $manifests in Drupal\format_strawberryfield\Plugin\Field\FieldFormatter\StrawberryBaseIIIFManifestFormatter->fetchIIIF() (line 381 of modules/contrib/format_strawberryfield/src/Plugin/Field/FieldFormatter/StrawberryBaseIIIFManifestFormatter.php).
@DiegoPino DiegoPino merged commit b028600 into 1.2.0 Oct 16, 2023
@DiegoPino DiegoPino deleted the ISSUE-368 branch October 16, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metadata Meta(l) data Twig Extension Make those flowers reach the sun
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant