- Issue created by @zniki.ru
- Assigned to mohd sahzad
- Issue was unassigned.
- ๐ฎ๐ณIndia abhishek_virasat
abhishek_gupta1 โ made their first commit to this issueโs fork.
- last update
10 months ago 23 pass, 24 fail - Status changed to Needs review
10 months ago 7:35am 23 January 2024 - ๐ฎ๐ณIndia abhishek_virasat
@Nikolay Shapovalov, I have fixed the issue. please kindly review the MR.
- Status changed to Needs work
10 months ago 7:53am 23 January 2024 - ๐ท๐บRussia zniki.ru
Thanks, suggested changes doesn't fix the issue.
Tests failed because there is no module youtube. - Assigned to bakop
- Status changed to Active
10 months ago 3:42pm 24 January 2024 - ๐ซ๐ทFrance bakop Lille
Hi, i will try to add the composer.json. The key is to know if the ci gonna take in count the file. If it's not the case, the maintainer will have to do a modification.
- Merge request !7Issue #3416385: YoutubeFieldTest::testYoutubeField Undefined array key "youtube" โ (Open) created by bakop
- last update
10 months ago 45 pass - Issue was unassigned.
- Status changed to Needs review
10 months ago 3:59pm 24 January 2024 - ๐ซ๐ทFrance bakop Lille
Add the composer.json solved this issue. You can review the MR!7
- Status changed to Needs work
10 months ago 8:23pm 24 January 2024 - ๐ท๐บRussia zniki.ru
Thank you, left a feedback.
During review found issue in README.md, issue create ๐ [novice] Change layout_fields to layout_paragraphs in README.md Active - ๐ซ๐ทFrance bakop Lille
Why you change the status ? Le problรจme est rรฉsolu et le README n'a rien ร voir avec les erreurs de tests
If you have nothing to said about the code, can you pass the issue in need of revision or review by the community?
Thanks in advance - ๐ท๐บRussia zniki.ru
Nikolay Shapovalov โ changed the visibility of the branch 3416385-youtubefieldtesttestyoutubefield-undefined-array to hidden.
- Assigned to bakop
- Status changed to Active
10 months ago 3:33pm 25 January 2024 - last update
10 months ago 45 pass - Issue was unassigned.
- Status changed to Needs review
10 months ago 3:53pm 25 January 2024 - ๐ซ๐ทFrance bakop Lille
I made the requested changes, you can review the MR.
- Status changed to RTBC
7 months ago 4:54pm 19 April 2024 - ๐ฌ๐งUnited Kingdom jofitz
All 4 unresolved threads have been addressed.
LGTMIt would be really useful to get this merged and released ASAP to help with test failures on other issues, e.g. ๐ It is no longer possible to add multiple field states Needs review