progressivetech / net.ourpowerbase.qrcodecheckin

QRCode Checkin allows you to send an email that contains a scanable code to the registered participants for your event.
Other
16 stars 15 forks source link

Getting "There was an error updating the status. Sorry." #28

Closed kathcarruthers closed 1 year ago

kathcarruthers commented 1 year ago

I'm logged in as admin (Drupal 9.5.8, CiviCRM 5.55.2 -- avoiding all but very necessary changes until after the Gala) and was worked out how to get my table numbers to show up without adding anything, but when I try to scan a code to Update to Attended, I get "There was an error updating the status. Sorry."

I've opened one on my computer to take a look at the error and see:

image

Any help or advice is very appreciated.

kathcarruthers commented 1 year ago

Not sure if this is helpful, but after clicking on the Update button:

image
jmcclelland commented 1 year ago

Thanks for the report! Unfortunately, I have not tested this on Drupal 9 and I'm pretty sure that's where the problem lay. We're in the process of working on our Drupal 9 upgrade, so I'll probably get to this in the next few months, but for now I think I can clearly see the problem and can at least point you in the right direction for the fix.

Here is where we try to extract the participant id.

In Drupal 7, your URL would be: /qrcheckin/18013/blahblah - but in Drupal 9, the slash is converted to %3A so you get: /qrcheckin/18013%3Ablahblah.

I'm not sure why it's converted or if that can be prevented, but perhaps the simplest solution is to add a new stanza to that check that will try to extract the participant id in a way that handles the "%3A" instead of the slash.

kathcarruthers commented 1 year ago

Thank you! I'll ask my maintenance folks if they can fix this, unless you would be willing to take some money/have the time available this week?

Worst case, we can check everyone in and direct them to their tables, then update statuses after the event (but we know that nobody ever goes back in and manually updates the statuses which is part of the need for automation).

Thank you as well for the extension. It has revolutionized the way we're doing registrations already (using that with discount codes sold per table means so much less backend work).

jmcclelland commented 1 year ago

Thanks @kathcarruthers for the report and @SARCASMOOO for the fix. I've updated the extension - it takes a day for these changes to make their way to the CiviCRM download area so please be on the lookout tomorrow and let me know if it does the trick or if further changes are needed.

SARCASMOOO commented 1 year ago

Hi @jmcclelland,

we tested the changes on the CiviCRM website noted in this issue and it worked as expected.

Thanks!

jmcclelland commented 1 year ago

Thanks for the comment. I made a mistake publishing it so it didn't go out after all. Hopefully it is properly corrected and you'll see a new release tomorrow.

Guydn commented 1 year ago

I'm on WordPress 6.3.1, CiviCRM 5.65 and I have the same issue

CA erreur Qrcode

the URL civicrm/qrcodecheckin/3/2e85a4b1e869f0052c0b62c60ff71cd4d60df7d16005a575117c0a5b10c8191e/

jmcclelland commented 1 year ago

@Guydn can you please click in the console on the left pointing triangle to expand the message and reveal what the error message says?

Guydn commented 1 year ago

Thanks a lot for your fast reply ! CA erreur code 2 does it help ?

jmcclelland commented 1 year ago

@Guydn Can you try replacing the api/v3/Qrcodecheckin/Checkin.php with this updated file and let me know if that works?

Guydn commented 1 year ago

Thanks a lot ! I let you know very soon. We are studying another option.