Skip to content

Conversation

@barnabasJ
Copy link
Contributor

No description provided.

@barnabasJ barnabasJ force-pushed the after-transaction-test branch from c096ef6 to 88e7c6b Compare December 5, 2025 20:06
@zachdaniel
Copy link
Contributor

@barnabasJ would you mind trying out removing this restriction to see if it works with AshPostgres? By my understanding it should be possible to do after_transaction calls with atomics, so it may be as easy as removing this restriction and adding some tests for it.

@barnabasJ
Copy link
Contributor Author

@barnabasJ would you mind trying out removing this restriction to see if it works with AshPostgres? By my understanding it should be possible to do after_transaction calls with atomics, so it may be as easy as removing this restriction and adding some tests for it.

I'm working on something, I'm trying to understand the whole atomic_after_action thing the regular Ash.Changeset.after_action and the phases around that to see if we need Todo something like that as well

1 similar comment
@barnabasJ
Copy link
Contributor Author

@barnabasJ would you mind trying out removing this restriction to see if it works with AshPostgres? By my understanding it should be possible to do after_transaction calls with atomics, so it may be as easy as removing this restriction and adding some tests for it.

I'm working on something, I'm trying to understand the whole atomic_after_action thing the regular Ash.Changeset.after_action and the phases around that to see if we need Todo something like that as well

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.

2 participants