Account created on 4 April 2023, about 2 years ago
#

Merge Requests

More

Recent comments

🇧🇪Belgium Jonasanne

What's the status here?

🇧🇪Belgium Jonasanne

#140 does not apply to 11.1.7 so uploaded a new diff of the MR.

🇧🇪Belgium Jonasanne

Can we get a D11 compatible release please?

🇧🇪Belgium Jonasanne

Can we get a merge and a d11 compatible release here please?

🇧🇪Belgium Jonasanne

This is my mistake.
I was to quick assuming it was a different issue but it's the fault of the D11 patch.
Closing this issue again since it's a duplicate.

🇧🇪Belgium Jonasanne

Can we get this merged and a D11 stable release?
We need this to make the RNG module D11 compatible.

🇧🇪Belgium Jonasanne

Created a MR with the necessary changes.
Can we get a merge and a new d11 compatible release?
Thanks

🇧🇪Belgium Jonasanne

I created a MR with the necessary changes.
Upgrade status still flags 1 deprecation but for D12 so we are safe in D11.
Added diff of MR to use as patch.

🇧🇪Belgium Jonasanne

I made a MR.
Can we get this merged and a D11 compatible release?
Thanks in advance.

🇧🇪Belgium Jonasanne

jonasanne changed the visibility of the branch 3521691-add-d11-support to hidden.

🇧🇪Belgium Jonasanne

jonasanne changed the visibility of the branch 3521565-add-d11-support to active.

🇧🇪Belgium Jonasanne

jonasanne changed the visibility of the branch 3521565-add-d11-support to hidden.

🇧🇪Belgium Jonasanne

jonasanne changed the visibility of the branch 3521565-add-d11-support to hidden.

🇧🇪Belgium Jonasanne

Created a MR.
Can we get this reviewed and merged for a d11 compatible release please?

Thanks!
Also added diff of MR to use as patch.

🇧🇪Belgium Jonasanne

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.

🇧🇪Belgium Jonasanne

jonasanne made their first commit to this issue’s fork.

🇧🇪Belgium Jonasanne

Has there been an update for a D11 ready release?

🇧🇪Belgium Jonasanne

Latest D11 patch does not apply

This one does:

🇧🇪Belgium Jonasanne

Latest D11 patch does not apply

This one does:

🇧🇪Belgium Jonasanne

What's the status here?
Can we get a D11 compatible release please?

🇧🇪Belgium Jonasanne

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.

🇧🇪Belgium Jonasanne

Uploading the diff as a patch here for ease of use.

🇧🇪Belgium Jonasanne

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.

🇧🇪Belgium Jonasanne

New patch against 2.x

🇧🇪Belgium Jonasanne

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?

🇧🇪Belgium Jonasanne

uploading diff of MR as patch here.

🇧🇪Belgium Jonasanne

tested and can cofirm it's ok.
Can we get a merge please?

🇧🇪Belgium Jonasanne

@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.

🇧🇪Belgium Jonasanne

I created a MR since monitoring has a D11 compatible version.
Can we get a merge and a new D11 ready release?
Thanks!

🇧🇪Belgium Jonasanne

Checked the module with upgrade_status and this MR would be enough.

Can we get a release please?

🇧🇪Belgium Jonasanne

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.

🇧🇪Belgium Jonasanne

Can we get a merge and a new release?
I tested the module with upgrade status and these changes look ok.

🇧🇪Belgium Jonasanne

Can we get a new release?
This has been merged but no release since.

🇧🇪Belgium 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!

🇧🇪Belgium Jonasanne

jonasanne made their first commit to this issue’s fork.

🇧🇪Belgium Jonasanne

uploading the diff of the MR here for easy usage as patch
works on 10.4.x

🇧🇪Belgium Jonasanne

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.

🇧🇪Belgium Jonasanne

This fix has been included in 10.3.x so I guess this issue can me closed and assumed fixed?

🇧🇪Belgium Jonasanne

I made a small mistake with the prev patch.
Goodmorning :D

🇧🇪Belgium Jonasanne

Added a diff from merge request for ease of use as patch.

Thankyou @scott_euser for updating this.

Production build 0.71.5 2024