🇮🇳India @Indranil Roy

Sorry @avpaderno, We will keep this in mind next time.
Can you please review the changes?

HI @simonbaese & avpaderno,
We have made the changes as per your comments. Please review the MR.
MR

Hi @mark shi,
The patch successfully solves the problem, but there's an issue when we click on 'Select all media.' It selects the media, but the message fails to update. Attaching screenshots for reference.
I'm currently working on this issue and will raise the MR soon.

Hi @sushyl,
I've retested the issue using the updated steps. The problem persisted when the integer value was 0. However, after applying your MR, the issue has been resolved. Attaching screenshots for reference.

Hi @sushyl,
I have checked on 8.x-1.x-dev version with drupal 10.2.6 & php 8.1.28 but not able to reproduce the issue. Attaching screenshots for reference.

Hi @Subhransu sekhar,
At the time of working on this feature request i also looked this issue as messenger block works differently with 9 &10 please refer to the attachments, So for bulk generate can we generate a file with the links?

I have tested the README.md file format and it seems proper to me, moving this to RTBC.

Hi,

I have tested the MR, and it is working fine for me. Here are the steps I followed:

1. Installed the module.
2. Applied the patch.
3. Installed the required libraries as described in the readme file.

After installing the libraries properly, the error message no longer appears on the status report page.

Thanks @simonbaese for the review.
For point 16 will you please guide us on what is expected or share some documentation that mentions what should be done in this case?

Hi purushotam.rai, I 've tested your patch it working as expected. I'm currently converting it to a MR.

Thanks!

Hi @vishal.kadam, I've changed the codebase as per your suggestion in #4 with the following commit.

Thanks !

Indranil Roy → created an issue.

Hi @rajdip_755, MR looks good to me.

Hi @kalash-j, addressed your comment#7 and updated the MR. Please check.

Hi @dzastin, Not able to reproduce the error in Drupal 10.1.6 with PHP 8.2.
Steps :
1. require the module.
2. enable the module.
3. clear the cache.
do I need to add any configuration?

Hey @Diwakar07,
I have tested your MR. LGTM.

Hi @Arijit Acharya,
I have checked your MR, and it's working fine.
Attaching the screenshot for further reference.

Hey @acbramley,
I have raised the MR changed the targeted branch 2.x. please review.

Indranil Roy → made their first commit to this issue’s fork.

Indranil Roy → changed the visibility of the branch 3450455-allow-multiple-replacement to active.

Indranil Roy → changed the visibility of the branch 3450455-allow-multiple-replacement to hidden.

Hey @bbrala, I have raised an MR please review.

Hey @Amandeep123, I have raised the pr please review.

Hey @Amandeep123, I am working on it.

Hey @endless_wander, I will work on it.

Hi @karavkars,
Thanks for your reply.
I have tested your patch again, able to reproduce the solution after saving the node page.
yes, it's working correctly.
After doing the step `Use "change number of rows/columns" to reduce rows from 5, to 4.` The row "1" which is now the footer at this point is removed. so working fine.

Hi @karavkars,
I have tested your patch it didn't work for me.
What I have done:
1. First added a table of 5*5.
2. Then reorder the table using drag and rebuild the table.
Notice the following issue.
1. Both fornt-end and back-end table row number didn't Updated. attaching the screenshots for ref:-

Hi, I have raised a MR please review.

Indranil Roy → changed the visibility of the branch 3250412-new-action-needed to hidden.

Thanks for the review @kunal_singh, Will surely look into it.

Thanks @kriboogh for your suggestion, will surely work on this.

Hi @kriboogh, I have raised the MR, Needs review.

Hi @kriboogh, I am working on this issue.

Hi I have raised the MR. Needs review.

Hi @3li, I'm working on this issue.

Indranil Roy → made their first commit to this issue’s fork.

@chaitanyadessai i have checked your code its working. thanks.

I have started working on this feature.

Hi @solideogloria, you can go to module uninstall page i.e /admin/modules/uninstall there is an option to remove extra data link where you can remove the data itself then go for uninstall the module.

Hi @geoanders, Its already merged. Can you please move it to the fixed(closed) state?

Hi @Mohd Sahzad I have tested your solution. it's working fine. I have attached screenshots.

Hey @prudloff, I have added the commit `3363915: Added 2FA setting page to admin theme.` and raised the MR please review.

Indranil Roy → made their first commit to this issue’s fork.

Hey @aman_lnwebworks, i am working on this issue.

I have implemented this feature.

Hey @geoanders I am currently working on this feature.

Indranil Roy → created an issue.

Hey @saltednut, I have added the patch please verify it.

Hi @grevil, I have checked the issue currently on this version releases have the proper README.md according to the markdown standard please check https://git.drupalcode.org/project/field_group/-/blob/8.x-3.x/README.md?...

Indranil Roy → created an issue.

+

Hi, I am currently working on this feature.

Indranil Roy → made their first commit to this issue’s fork.

Indranil Roy → made their first commit to this issue’s fork.

Indranil Roy → made their first commit to this issue’s fork.

@KapilV can you please add some issue descriptions?

I have created an MR please check.

Indranil Roy → made their first commit to this issue’s fork.

Indranil Roy → created an issue.

I am using Drupal version 10.0.11-dev i am not getting this error after i install this module may be this issue is resolved with the Drupal releases.

I have checked your issue in my case there was no .git directory in the root after install this module.

Indranil Roy → made their first commit to this issue’s fork.

I have added the patch.

Indranil Roy → made their first commit to this issue’s fork.

I have added the patch.

Indranil Roy → made their first commit to this issue’s fork.

Indranil Roy → made their first commit to this issue’s fork.