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.

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

Identify profiles by ID #73

Merged
merged 10 commits into from
Apr 5, 2024

Conversation

MarcMichalsky
Copy link
Contributor

Storing profiles in a separate db table instead of native CiviCRM settings was implemented in fd47b91.

Nevertheless, the name of the profile is still used to identify it. This leads to problems like #70.

I have now implemented the identification of profiles by their database ID.

This PR depends on PR #72 because it uses the custom ProfileException class!

CRM/Twingle/Form/Profile.php Outdated Show resolved Hide resolved
CRM/Twingle/Form/Profile.php Outdated Show resolved Hide resolved
CRM/Twingle/Profile.php Show resolved Hide resolved
CRM/Twingle/Profile.php Outdated Show resolved Hide resolved
CRM/Twingle/Profile.php Outdated Show resolved Hide resolved
@jensschuppe jensschuppe added this to the 1.5 milestone Aug 16, 2023
@jensschuppe jensschuppe added enhancement New feature or request status:needs work There is code, but it needs additional work before it should be reviewed labels Aug 16, 2023
@jensschuppe
Copy link
Collaborator

@MarcMichalsky I rebased onto current master which has the required changes from #72 - could you have a look whether you can spot anything odd? There were some merge conflicts I resolved but I would like to get another pair of eyes on it before merging …

@jensschuppe jensschuppe added status:needs review Code needs review and testing and removed status:needs work There is code, but it needs additional work before it should be reviewed labels Mar 25, 2024
@MarcMichalsky
Copy link
Contributor Author

I'll have a look at it next week. 🏝️

@jensschuppe jensschuppe merged commit aa2c938 into systopia:master Apr 5, 2024
0 of 10 checks passed
@jensschuppe jensschuppe added status:fixed The issue has been resolved (usually by committing/merging code) and removed status:needs review Code needs review and testing labels Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request status:fixed The issue has been resolved (usually by committing/merging code)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants