-
Notifications
You must be signed in to change notification settings - Fork 77
Hanami integration #110
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
Open
swilgosz
wants to merge
20
commits into
RailsEventStore:master
Choose a base branch
from
swilgosz:hanami-integration
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Hanami integration #110
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
350593f
Add initial hanami application repo
9afec8f
Rename the Main slice to Crm
swilgosz fe46796
Add dependencies to gemfile
swilgosz 02dbb30
Register command_bus and dummy event_repository
swilgosz 51c8c26
Fix missing required file
swilgosz 74bea0e
Fix invalid dependencies in infra gem
swilgosz 8dffb24
Add create_events migration
swilgosz 242a4aa
Add missing dry-* libraries requires in infra
swilgosz 1b5719e
Configure ROM repository to work with Hanami application
swilgosz 3e14c60
Add action to create customer in CRM context
swilgosz 11041c6
TMP: disable everything from ecomerce except CRM for easier integrati…
swilgosz 4c10e22
Disable assets server - not used for now
swilgosz 798face
Update README to include correct setup instructions
swilgosz 54681e8
Fix invalid redirect path
swilgosz 3e7bb1d
Switch to forked RubyEventStore-ROM
swilgosz 677baa0
Integrate CRM context
swilgosz 6f83529
Restore general ecommerce configuration
swilgosz 6de6008
Update deps
swilgosz e0f836c
Sequel needs JSON strings, as opposed to ActiveRecord
mostlyobvious 9702bbf
Merge branch 'master' into hanami-integration
mostlyobvious File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,4 @@ | ||
require "active_support/isolated_execution_state" | ||
require "active_support/core_ext/hash" | ||
|
||
module Infra | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason to include this? It has been already reverted once so I'd like to see the backtrace why it reappears.