Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adjust for using CiviCRM contact checksum instead of contact hash #7

Open
jensschuppe opened this issue Jul 7, 2021 · 1 comment · May be fixed by #27
Open

Adjust for using CiviCRM contact checksum instead of contact hash #7

jensschuppe opened this issue Jul 7, 2021 · 1 comment · May be fixed by #27
Labels
enhancement New feature or request status:needs review Code needs review and testing
Milestone
1.0

Comments

@jensschuppe
Copy link
Collaborator

jensschuppe commented Jul 7, 2021 •
edited
Loading

Followup to systopia/de.systopia.newsletter/issues/11

The extension uses CiviCRM contact hashes and should use contact checksums instead (because they expire, whereas contact hashes don't) and this module will need to be adjusted for correctly implementing this change.

This is a blocker for a stable 1.0/7.x-1.0 release.
@jensschuppe jensschuppe added bug Something isn't working help wanted Extra attention is needed 7.x labels Jul 7, 2021
@jensschuppe jensschuppe added this to the 1.0 milestone Jul 7, 2021
@dontub
Copy link
Contributor

dontub commented Jun 22, 2023

Because of changes in the API through systopia/de.systopia.newsletter#21 every appearance of contact_hash should be replaced by contact_checksum.

@jensschuppe jensschuppe mentioned this issue Oct 19, 2023
Closed
@jensschuppe jensschuppe added status:needs review Code needs review and testing enhancement New feature or request and removed help wanted Extra attention is needed needs funding bug Something isn't working labels Oct 19, 2023
@jensschuppe jensschuppe linked a pull request Aug 12, 2024 that will close this issue
Open
@jensschuppe jensschuppe linked a pull request Aug 12, 2024 that will close this issue
Replace occurrences of "hash" with "checksum". #27
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request status:needs review Code needs review and testing
Projects
None yet
Milestone
1.0
2 participants
@dontub @jensschuppe