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

findActorOrCreate was changed to search by apPublicUrl which is wrong, revert to apProfileId #870

Merged
merged 3 commits into from
Jun 28, 2024

Conversation

BentiGorlich
Copy link
Member

Change was made by me in #853

@BentiGorlich BentiGorlich added bug Something isn't working activitypub ActivityPub related issues backend Backend related issues and pull requests high priority critical issue or PR that impacts production labels Jun 28, 2024
@BentiGorlich BentiGorlich requested a review from a user June 28, 2024 07:05
@BentiGorlich BentiGorlich self-assigned this Jun 28, 2024
@BentiGorlich
Copy link
Member Author

Should I revert the migration adding a unique index over apPublicUrl?

@BentiGorlich BentiGorlich merged commit c209878 into main Jun 28, 2024
7 checks passed
@BentiGorlich BentiGorlich deleted the fix/findActorOrCreate-using-wrong-field branch June 28, 2024 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
activitypub ActivityPub related issues backend Backend related issues and pull requests bug Something isn't working high priority critical issue or PR that impacts production
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant