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

Upsert a contact when seeing it for the first time #211

Merged
merged 4 commits into from
Nov 20, 2023

Conversation

gferon
Copy link
Collaborator

@gferon gferon commented Nov 19, 2023

Closes #98 and #171

Also fixes updating profile keys (tentatively added a few commits ealier) which was in the wrong place (silly giant match statements).

@gferon gferon force-pushed the save-contact-on-profile-change branch from 142a865 to 30f06bc Compare November 20, 2023 09:06
@gferon gferon changed the title Upsert a contact on profile key change Upsert a contact on profile key first time/change Nov 20, 2023
@gferon gferon changed the title Upsert a contact on profile key first time/change Upsert a contact when seeing it for the first time Nov 20, 2023
@gferon gferon marked this pull request as ready for review November 20, 2023 09:18
@gferon gferon added this to the 0.6.0 milestone Nov 20, 2023
@gferon gferon merged commit 676cf3a into main Nov 20, 2023
4 checks passed
@gferon gferon deleted the save-contact-on-profile-change branch November 20, 2023 09:21
@gferon
Copy link
Collaborator Author

gferon commented Nov 20, 2023

Thanks to @nanu-c for the initial inspiration for this PR! (#87)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unknown contact handling
1 participant