- Issue created by @DamienMcKenna
- Status changed to Needs review
12 months ago 11:30am 11 January 2024 - last update
12 months ago 2 fail - 🇺🇸United States DamienMcKenna NH, USA
Let's see if this solves it, or if there are more problems.
The last submitted patch, 2: views_rss-n3414012-2.patch, failed testing. View results →
- Status changed to Needs review
12 months ago 4:02pm 11 January 2024 - last update
12 months ago 2 fail - 🇮🇳India sidharth_soman Bangalore
DamienMcKenna → credited sidharth_soman → .
- 🇺🇸United States DamienMcKenna NH, USA
Adding sidharth_soman who worked on a subset of this in #3412474.
The last submitted patch, 4: views_rss-n3414012-4.patch, failed testing. View results →
- Status changed to Needs review
12 months ago 4:28pm 11 January 2024 - last update
12 months ago 2 fail The last submitted patch, 11: views_rss-n3414012-10.patch, failed testing. View results →
- 🇺🇸United States DamienMcKenna NH, USA
For some reason the content type header of "application/rss+xml" is not being set. I'm not sure why this changed in 10.2 whereas it worked in 10.1.
- 🇪🇸Spain budalokko Girona
Core added the following in the
render
function of their Rss feed style plugin:'#attached' => [ 'http_header' => [ ['Content-Type', 'application/rss+xml; charset=utf-8'], ], ],
This was the issue -> https://www.drupal.org/project/drupal/issues/3272985 🐛 RSS Feed header reverts to text/html when cached Fixed
Applying the same change to 8.x-2.x fixes the content type header issue. I can provide as a patch, but should it be in this issue or a new one @DamienMcKenna?
- 🇺🇸United States DamienMcKenna NH, USA
Slightly expanding the scope, because so many things are interconnected.
- Status changed to Needs review
11 months ago 11:36am 7 February 2024 - last update
11 months ago 2 fail The last submitted patch, 17: views_rss-n3414012-17.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Assigned to DamienMcKenna
- Open on Drupal.org →Core: 10.2.x + Environment: PHP 8.2 & MySQL 8last update
7 months ago Waiting for branch to pass - last update
7 months ago Patch Failed to Apply - Status changed to Needs review
7 months ago 5:36pm 3 June 2024 - Open on Drupal.org →Core: 10.2.x + Environment: PHP 8.2 & MySQL 8last update
7 months ago Waiting for branch to pass The last submitted patch, 21: views_rss-n3414012-21.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Issue was unassigned.
- 🇺🇸United States DamienMcKenna NH, USA
I'm not in a position to keep working on this at the moment, might someone else be able to continue the work?
- 🇺🇸United States DamienMcKenna NH, USA
Thank you for the offer - if you're able to resolve this I'll consider it.
- ivnish Kazakhstan
Can you commit .gitlab-ci.yml ? We can use MR with CI pipelines
- Merge request !7Resolve #3414012 "Fix tests, module for Drupal 10.2 & PHP 8.2" → (Merged) created by ivnish
- Open on Drupal.org →Core: 10.2.x + Environment: PHP 8.2 & MySQL 8last update
7 months ago Waiting for branch to pass - Open on Drupal.org →Core: 10.2.x + Environment: PHP 8.2 & MySQL 8last update
7 months ago Waiting for branch to pass - Status changed to Needs review
7 months ago 11:04am 8 June 2024 - ivnish Kazakhstan
@Damien please review. Pipeline is here https://git.drupalcode.org/issue/views_rss-3414012/-/pipelines/194294
I don't fix cspell, phpcs and phpstan issues. I think it's a task for another issue
- last update
7 months ago 2 pass -
DamienMcKenna →
committed 47f9d90a on 8.x-2.x authored by
ivnish →
Issue #3414012 by DamienMcKenna, ivnish, sidharth_soman: Fix tests,...
-
DamienMcKenna →
committed 47f9d90a on 8.x-2.x authored by
ivnish →
- Status changed to Fixed
7 months ago 2:48pm 11 June 2024 - 🇺🇸United States DamienMcKenna NH, USA
Committed. Thank you!
And ivnish, welcome to the team!
Automatically closed - issue fixed for 2 weeks with no activity.