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

fix: delete tombstones and log failing activities #98 #99

Merged
merged 1 commit into from
Jan 29, 2025
Merged

Conversation

fauno
Copy link
Collaborator

@fauno fauno commented Jan 28, 2025

No description provided.

@fauno fauno self-assigned this Jan 28, 2025
let object = activity.object
if (typeof object !== 'string') {
if ((object != null) && 'id' in object && typeof object.id === 'string') {
object = object.id
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we have a helper to obtain the object id?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I think we could stand to have a bunch of helpers, not sure when to fit it in

Copy link
Contributor

@RangerMauve RangerMauve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, ty!

let object = activity.object
if (typeof object !== 'string') {
if ((object != null) && 'id' in object && typeof object.id === 'string') {
object = object.id
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I think we could stand to have a bunch of helpers, not sure when to fit it in

@RangerMauve RangerMauve merged commit 9dec86f into main Jan 29, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

2 participants