Uploaded diff of MR as patch for ease of use.
Can we get a D11 compatible release version please?
Can we get a D11 compatible release please?
rerolled patch to 3.0.0-beta1
Now I don't have time to fix the other tests so if someone else wants to pick this up that would also be awesome.
jonasanne → made their first commit to this issue’s fork.
borisson_ → credited jonasanne → .
bramdriesen → credited jonasanne → .
patch reroll
Uploaded MR diff as patch for ease of use
Has there been an update for a D11 ready release?
borisson_ → credited jonasanne → .
Can we get a release here?
jonasanne → created an issue.
rerolled patch to 3.0.0-alpha4
jonasanne → created an issue.
Latest D11 patch does not apply
This one does:
Latest D11 patch does not apply
This one does:
bramdriesen → credited jonasanne → .
What's the status here?
Can we get a D11 compatible release please?
jonasanne → created an issue.
We are having the same issue where results are 0 and that the render is not working correctly.
I tested this MR and I can confirm this works.
also uploaded diff of MR as patch.
bramdriesen → credited jonasanne → .
Uploading the diff as a patch here for ease of use.
Can we get a merge and release please?
jonasanne → made their first commit to this issue’s fork.
Aleix are u sure the answer above is on the correct ticket?
I don't understand why you closed this ticket to fixed and the geofield_map require is still inside the compose.json in the new release.
added mr as patch
Nevermind patch does not apply after change to target branch 2.0.x instead of 2.x.
I'm also confused. What's the difference here and why these branch names?
uploading diff of MR as patch here.
tested and can cofirm it's ok.
Can we get a merge please?
@flyke
When I check git.drupalcode.org and change the target branch to 2.0.x, it does not say anything about a merge conflict.
Can you give me some more information about the warnings?
Drupal version, steps to reproduce, and whether you see this warning while using these changes or without them.
rerolled patch #49 to 3.0.x
Can we get a merge and a D11 ready release please?
I created a MR since monitoring has a D11 compatible version.
Can we get a merge and a new D11 ready release?
Thanks!
jonasanne → created an issue.
+1 for merge please.
Checked the module with upgrade_status and this MR would be enough.
Can we get a release please?
bramdriesen → credited jonasanne → .
jonasanne → created an issue.
I converted Jelle his changes into a MR and added the change in monitoring.schema.yml.
Also uploaded the diff of the MR as patch here.
jonasanne → made their first commit to this issue’s fork.
Can we get a merge and a new release?
I tested the module with upgrade status and these changes look ok.
borisson_ → credited jonasanne → .
Can we get a new release?
This has been merged but no release since.
bramdriesen → credited jonasanne → .
I have tested this MR and can confirm that it fixes the problem.
Could we get a new release as soon as possible, please?
I’ve also resolved the merge conflict and included a diff of the merge here for convenience as a patch. Additionally, I’ve removed the comments from the libraries file, as they are not necessary to retain.
Thank you!
jonasanne → made their first commit to this issue’s fork.
borisson_ → credited jonasanne → .
borisson_ → credited jonasanne → .
bramdriesen → credited jonasanne → .
uploading the diff of the MR here for easy usage as patch
works on 10.4.x
reroll patch and uploaded it here for ease of use
I rerolled the patch from @zero2one to 10.3.10 and made a MR for this.
Also gonna upload the MR diff here as a patch for ease of use.
jonasanne → made their first commit to this issue’s fork.
pjonckiere → credited jonasanne → .
borisson_ → credited jonasanne → .
This fix has been included in 10.3.x so I guess this issue can me closed and assumed fixed?
Uploaded the diff of the MR for ease of use as a patch
I made a small mistake with the prev patch.
Goodmorning :D
Added a diff from merge request for ease of use as patch.
Thankyou @scott_euser for updating this.
pjonckiere → credited jonasanne → .
borisson_ → credited jonasanne → .
borisson_ → credited jonasanne → .
borisson_ → credited jonasanne → .
Rerolled #93
Reroll patch from #2 for 2.0.2
I accidentally uploaded the wrong patch.
This one works correctly for me at 10.3.2
Rerolled patch for 10.3.2
I Replace deprecated watchdog_exception() with \Drupal\Core\Utility\Error::logException()
- Inject logger service into the File class.
- Update logResults method to use \Drupal\Core\Utility\Error::logException() for logging exceptions.
- Ensure proper dependency injection and autoloading configuration.
PHPUNIT testing still fails and needs to be checked
We also need to wait on a new D11 ready release of drupal/monitoring
Jonasanne → made their first commit to this issue’s fork.
+1
Can we please have a new release D11 compatible?
Kind regards,
Hi Dieter,
I tested everything this morning, and it all seems to be working well in one of our projects. I've also created a merge request.
If there's anything else I need to do, please let me know.
Best regards,
I have made the summarized changes. However, it still needs to be tested with a valid Flexmail account since I don't have one at the moment.
I did the changes and tested it with upgrade status.
Jonasanne → created an issue.
Uploaded patch here for ease of use
I have tested the MR and I can confirm this resolves the problem and the error is gone.
Jonasanne → made their first commit to this issue’s fork.
I tested this on 10.2.6 with php 8.1 (branch 2.0.x and branch 3.0.x) and on 10.3.0 with 8.3 while following the steps above and I do not have the error.
Looks great to me!
Jonasanne → created an issue.
I tested this PR inside configuration inspector and it looks good.
Jonasanne → changed the visibility of the branch CozyDrupalNerd-2.0.x-patch-a239 to active.
Jonasanne → changed the visibility of the branch CozyDrupalNerd-2.0.x-patch-a239 to hidden.
Jonasanne → created an issue.
Jonasanne → created an issue.
I updated the file with the correct naming conventions and tested it in config inspector.