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

Define Rails.configuration.event_store after initialization #18

Merged
merged 2 commits into from
Apr 26, 2024

Conversation

Samsinite
Copy link
Contributor

@Samsinite Samsinite commented Apr 25, 2024

This commit updates ActiveEventStore to define Rails.configuration.event_store after the event store has been initialized. Fixes #17

What is the purpose of this pull request?

Defines Rails.configuration.event_store after the event store is initialized, so that the rails event store browser tools (and other tools) should ideally just work out of the box. Fixes #17

Checklist

  • I've added tests for this change
  • I've added a Changelog entry
  • I've updated a documentation

This commit updates ActiveEventStore to define Rails.configuration.event_store after the event store has been initialized. Fixes palkan#17
@Samsinite
Copy link
Contributor Author

Samsinite commented Apr 25, 2024

Any idea what is wrong with sqlite on the 3.0, 3.1, and 3.2 builds? Guessing it is related to the tests using the 2.X release 3.0+, since it doesn't run on Ruby 2.7.

Edit: Figured it out, looks like active record pinned it to ~> 1.4, there appears to be an update for 7.1 to run on 2.x, but it hasn't been released yet I believe.

@palkan palkan force-pushed the master branch 3 times, most recently from 8895b41 to 8c6f78e Compare April 26, 2024 18:21
@palkan
Copy link
Owner

palkan commented Apr 26, 2024

Looks good 👍 Thanks!

P.S. I fixed the sqlite/Rails 8 problems in master; should be green after the merge.

@palkan palkan merged commit b89082d into palkan:master Apr 26, 2024
9 of 16 checks passed
@Samsinite Samsinite deleted the patch-1 branch April 27, 2024 01:37
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.

RailsEventStore::Browser doesn't work out of the box
2 participants