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

Sync base gem #6

Open
wants to merge 5 commits into
base: core
Choose a base branch
from
Open

Sync base gem #6

wants to merge 5 commits into from

Conversation

Hammam94
Copy link

@Hammam94 Hammam94 commented Jul 6, 2023

No description provided.

bwilkins and others added 5 commits April 14, 2023 10:35
* Allow paper_trail-background to work with newer versions of paper_trail

* Only require version file

* Work with newer versions of activerecord

* Ensure RecordTrail exists when this gem is loaded

* This is already done by the Events; this in particular was breaking out STI model versions.

* Use ActiveJob semantics instead of specifying Sidekiq

* Version bump

* Remove "Sidekiq" references, adapting to ActiveJob instead

* Use string name, as `version_class` is a Class object now

* Use ActiveRecord::Base instead of the version class

* Add RSpec helpers/support

* Use correct module names!

* Put rspec in correct location for being able to require it from outside the gem

* Extract the normally open transactions to an accessible location; needs a better home than a default on the interface, though.

* require after_transaction
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.

4 participants