- Issue created by @gidarai
- Status changed to Needs review
over 1 year ago 12:16pm 29 August 2023 - last update
over 1 year ago Build Successful The last submitted patch, 2: 3384000-2.patch, failed testing. View results →
- 🇳🇱Netherlands gidarai
Changed status back to "needs review" as the code errors aren't related to this patch
@gidarai, the text is changed on the configuration, but the icon is still showing the same.
- 🇮🇳India snehal-chibde
Hello, applied patch #2. It is working as per requirement. Here are before and after screenshots for the same. RTBC++.
- Open on Drupal.org →Core: 10.1.x + Environment: PHP 8.1 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇳🇱Netherlands gidarai
@Tanuja Bohra Yes i was working on the icons for the frontend. Here is the patch including the frontend icon changes.
- Status changed to RTBC
over 1 year ago 7:25am 6 September 2023 - Status changed to Needs work
over 1 year ago 6:59pm 8 September 2023 - 🇺🇸United States shelane
The icons on the block configuration page still have the twitter bird. Also, there was a switch in the order of services in the template. That is beyond the scope of this issue. It's fine if you want to copy the template to your theme to change the order, but it shouldn't be changed here.
- Status changed to Fixed
over 1 year ago 6:19pm 15 September 2023 - 🇺🇸United States shelane
I update the config pages and also changed the config values of the service from twitter to x. I included an update hook to update existing config.
Automatically closed - issue fixed for 2 weeks with no activity.