After discussing this with some folks and security team members we are clear on why service accounts can't apply here. I am moving this to Close (works as designed) as we don't need to move forward with this at this time.
Created a MR based on the above patch. thought it might make getting this into a release a little easier.
johnpicozzi → made their first commit to this issue’s fork.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
jdleonard → credited johnpicozzi → .
I added Providence as a location and adding Group & Camp
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
I would agree this can be closed. I'm moving this to Fixed and giving credit to all involved.
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.
seantwalsh → credited johnpicozzi → .
Bumping this issue again. Looking to try to get the new version up before DrupalCon.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
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.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
I was here :-)
mandclu → credited johnpicozzi → .
seantwalsh → credited johnpicozzi → .
seantwalsh → credited johnpicozzi → .
seantwalsh → credited johnpicozzi → .
froboy → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
Closing this and adding the speakers to the speaker issue
johnpicozzi → created an issue.
johnpicozzi → created an issue.
johnpicozzi → created an issue.
johnpicozzi → created an issue.
johnpicozzi → created an issue.
johnpicozzi → created an issue.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
Ok, MR is now mergeable. Would be great to get this rolled in.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
froboy → credited johnpicozzi → .
@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.
Stilling having this issue on beta1- upping the version. The patches above don't apply cleanly.
This change is simple enough and the patch applies cleanly. I'm moving this to RTBC and suggest a release is tagged.
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
seantwalsh → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
Closing this in favor of #3467917: Rethink the Event Categories →
I was here!
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
nicxvan → credited johnpicozzi → .
@Liam Morland I'm seeing that now. I have tested this locally and it works for me. RTBC +1
@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.
Added the contents of the patch in #4 as an MR. The MR needs testing and review. Moving this to needs review.
johnpicozzi → made their first commit to this issue’s fork.
@chrisfromredfin This has the changes we talked about and it passes all tests with no merge conflicts. Ready to go!