Providence, RI
Account created on 14 October 2010, over 14 years ago
#

Merge Requests

Recent comments

🇺🇸United States johnpicozzi Providence, RI

I added Providence as a location and adding Group & Camp

🇺🇸United States johnpicozzi Providence, RI

I would agree this can be closed. I'm moving this to Fixed and giving credit to all involved.

🇺🇸United States johnpicozzi Providence, RI

Closing this issue as outdated as the 1.x version of this module only supports Drupal 8 & 9. We are in the works of releasing a 2.x version of the module that will support current versions of Drupal.

🇺🇸United States johnpicozzi Providence, RI

Bumping this issue again. Looking to try to get the new version up before DrupalCon.

🇺🇸United States johnpicozzi Providence, RI

We are moving forward posting our 2.0 version of this module and I need my co-worker added. Could you please also add Alexey Korepov (murz) as a co-maintainer on this project.

🇺🇸United States johnpicozzi Providence, RI

Closing this and adding the speakers to the speaker issue

🇺🇸United States johnpicozzi Providence, RI

Ok, MR is now mergeable. Would be great to get this rolled in.

🇺🇸United States johnpicozzi Providence, RI

I was having the issue noted in #33 above. The patch in #39 resolved the issue. I would suggest moving that patch to an MR and getting it rolled in ASAP.

I will try to create an MR later if time permits. Thanks

🇺🇸United States johnpicozzi Providence, RI

@it-cru Thanks for the assist here. After looking at my cloudflare.settings.yml and comparing with yours I noticed a difference. After re-saving my Cloudflare settings the issues has gone away. I'm moving this to closed (outdated) as I think this has been resolved.

🇺🇸United States johnpicozzi Providence, RI

Stilling having this issue on beta1- upping the version. The patches above don't apply cleanly.

🇺🇸United States johnpicozzi Providence, RI

This change is simple enough and the patch applies cleanly. I'm moving this to RTBC and suggest a release is tagged.

🇺🇸United States johnpicozzi Providence, RI

@Liam Morland I'm seeing that now. I have tested this locally and it works for me. RTBC +1

🇺🇸United States johnpicozzi Providence, RI

@Liam Morland Looks like the MR doesn't pass all it's tests. I'll be reviewing the patch to confirm it resolves the issue shortly.

🇺🇸United States johnpicozzi Providence, RI

Added the contents of the patch in #4 as an MR. The MR needs testing and review. Moving this to needs review.

🇺🇸United States johnpicozzi Providence, RI

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

🇺🇸United States johnpicozzi Providence, RI

@chrisfromredfin This has the changes we talked about and it passes all tests with no merge conflicts. Ready to go!

🇺🇸United States johnpicozzi Providence, RI

@ben.hamelin Talked with @chrisfromredfin about this and the last update we would like to see is changing the alt to "Security note" for the img. Once this is done it should be good to go! Thanks for all the help here!

🇺🇸United States johnpicozzi Providence, RI

Updating the title of this issue and moving priority to Major as it makes the body field unusable.

🇺🇸United States johnpicozzi Providence, RI

I can confirm I also have this error with OpenAI CKEditor integration Module enabled. As soon as the module is disabled the CKEditor displays as expected. Prior to disabling the module the body filed on my content type would not display. See below screenshot

🇺🇸United States johnpicozzi Providence, RI

After review this looks and works the way I would expect and resolves this issues original point. Moving to RTBC! Thanks all!

Production build 0.71.5 2024