Account created on 15 November 2022, about 2 years ago
#

Merge Requests

More

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi @smustgrave,

I have checked this issue on versions 9.4.0-dev, 10.0.12-dev, 11.x-dev, but it is not reproducible on my end. I have attached screenshots for your reference.



I have followed all the steps mentioned here.

  • Setup Drupal
  • Activate Claro theme and set default theme
  • Create node with long title
  • Go to incognito tab with created node view page
  • There line-height between breadcrumbs and title is not properly showing And Node title and header padding not properly showing.

Please let me know what should be the next step from my side.

Thank you!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

I tried to check the issue with steps mentioned in issue description on Drupal 11, but unable to replicate the issue.

Below I have attached screenshots on desktop and mobile view. As, if the issue is not reproducible on latest version can we close this case?

I am moving this to Needs Review.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

Changed title from 'Do not require a 'title' field' to 'Remove 'not null' constraint from node title field/column.('Title' field present in node entity should not be mandatory.)'.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

While going through issue, got reference of linked issue ( https://www.drupal.org/project/drupal/issues/3347648 ๐Ÿ“Œ Remove @todo-comment linking to closed issue in layout-builder.css Closed: duplicate ).

There we have suggession to remove only @todo-comment linking. So removed only comment line and raised merge request(https://git.drupalcode.org/project/drupal/-/merge_requests/9763) for that.

@smustgrave As there is no UI change or any such change do we need to update summary? Please let me know.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

supriyak โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

MR has been created !10005(https://git.drupalcode.org/project/drupal/-/merge_requests/10005).

Written testcase as per requirement. Please let me know incase of any new changes needs to be add.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi @anybody,

Below are my findings and observations while working on this issue.

1. After enabling 'List.js' and 'Listjs views' modules when I try to edit any view and changed 'Format' to 'Listjs', it is giving me below error.
=> Error: Call to a member function __toString() on string in Drupal\listjs_views\Plugin\views\style\Listjs->buildOptionsForm() (line 74 of /var/www/html/modules/contrib/listjs/modules/listjs_views/src/Plugin/views/style/Listjs.php).
This error got resolved after removing '->__toString();' from line number 74.

Will you please let me know that can we create another issue regarding this error?

2. I have used 'Listjs' format in 'admin/content' view where it seems to be fuzzy search is working already. Below I have attached screen recording for it.

Will you please let me know that do we need to keep this issue open or we can close this? Or am I missing/misunderstood this issue? Please let me know so that I will get clarity on this and can work on it.

Thank you!!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

supriyak โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

After patch application still there are so many errors/warnings. So, moving this to 'Needs Work'.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi @sarwan_verma,

After patch apply I am getting below error, here I have attached screenshot FYI. Moving this to needs work.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

Please find below before and after patch screenshots. Before applying patch conditions were not working and after applying patch those are working. RTBC from my end.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

I have executed generate-theme command using starterkit. Below are the screenshots of template files before and after patch. We can see here 'description display before/after conditionals' in after-patch screenshot.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

I have executed generate-theme command using starterkit. Below are the screenshots of template files before and after patch. We can see here 'description display before/after conditionals' in after-patch screenshot.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Thank you @Anybody for reply.

Can you please check 'WatchdogMailerSettingsTest.php'? In this file there are no levels and sublevels, here they maintain sequence only.
So it will be confusing here. Please suggest on this.

I have done with my code just waiting for your reply on this.

Thanks!!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi @Anybody,

"enabled" key is already present in watchdog_mailer.settings for another field. It is affecting to changing key for 'watchdog_mailer_enable'.
Please suggest on this.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

supriyak โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

supriyak โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

I have tested this from my end. Moving this issue in RTBC.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi @DamienMcKenna,

I think this theme is not compiled yet.

I have tried to compile the files here. The data in table.css and table.scss is different.
So when I try to compile table.scss, previous data of that file got removed and only new changes are there which is not acceptable.

Please suggest me should I add the code in both the files?

Thanks!!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi @DamienMcKenna,

Thank you for your reply. I have pushed my new changes in same MR.
Please let me know your feedback on this.

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi Everyone,

This bug is occurring in 'Bootstrap Barrio Subtheme'.
To fix this bug I have raised MR : https://git.drupalcode.org/project/bootstrap_barrio/-/merge_requests/83. Please review it.

Thank you!!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

supriyak โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi @21kPiyush,

I have tried to reproduce the alert message on my local setup, but I am not getting it.
As you said when I am editing the webform submission the selected Date and time slot is coming in webform_booking element but it is not selected on calendar.

So we can create the task to do this functionality. Please let me know still you getting same alert message? And if yes let me know steps to reproduce it.

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Hi,

Applied patch successfully which is commented on '#2'. Issue is resolved using this patch. Attaching after and before screenshots.

@Muskan0215 Instead of using external CSS have you checked any other solution here?

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

Tested patch '#2' and 'MR !8' in #6.

Checking patch docroot/modules/contrib/email_confirmer/email_confirmer.info.yml...
Checking patch docroot/modules/contrib/email_confirmer/email_confirmer_user/email_confirmer_user.info.yml...
Applied patch docroot/modules/contrib/email_confirmer/email_confirmer.info.yml cleanly.
Applied patch docroot/modules/contrib/email_confirmer/email_confirmer_user/email_confirmer_user.info.yml cleanly.

Both the patches are successfully applying and changes are reflected.
Please find below before and after screenshots for same.
RTBC from me.

๐Ÿ‡ฎ๐Ÿ‡ณIndia supriyak

I am migrating some sites from D9 to D10 for that I need to make content_sync compatible with D10.
I tried to apply #9, #20 and #22 patches in my code, but can not make it success. So, creating another one.
Here is a patch based on status_upgrade checks and previous patches here.
It is for 3.0.0-beta1 version.

Production build 0.71.5 2024