Skip to content
This repository was archived by the owner on Oct 27, 2025. It is now read-only.

References many default#323

Draft
ryanwjackson wants to merge 7 commits into
masterfrom
rj/references_many_default
Draft

References many default#323
ryanwjackson wants to merge 7 commits into
masterfrom
rj/references_many_default

Conversation

@ryanwjackson

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread lib/ledger_sync.rb
require 'faraday'
require 'tempfile'
require 'pd_ruby'
require 'pry'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Need to remove

resource_class: resource_class,
value: value
)
# rescue Error::TypeError::ValueClassError

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Uncomment

@ryanwjackson

Copy link
Copy Markdown
Contributor Author

@jozefvaclavik this is more in line with what I'm thinking for defaults, but I have one failing coverage test.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant