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
Fix user.owned_events example in readme #44
base: master
Are you sure you want to change the base?
Conversation
Updates the readme to remove references to the deprecated `user.owned_events`, replaces with examples of how to fetch organization events
@@ -5,6 +5,7 @@ class User < Resource | |||
# NOTE: This name is pretty legacy. We should consider renaming | |||
# to "orders" to normalize things. | |||
has_many :owned_event_orders, object_class: 'Order', key: :orders | |||
has_many :organizations, object_class: 'Organization', key: :organizations |
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.
This looks good but is untested functionality. If you could add tests to cover this we can approve and merge it. You'll find examples of tests for associated relationships in a handful of places like https://github.com/eventbrite/eventbrite-sdk-ruby/blob/master/spec/eventbrite_sdk/organization_spec.rb#L25
9d548fd
to
b401e70
Compare
As noted in #46 the examples in the README don't work, so the library is not usable for really basic tasks (just want to list all the events owned by a user). This PR looks like its intended to fix that, but is stuck in limbo? |
I have encountered the same problem and happy to lend a hand to continue working on this PR and fix the problem if the owner doesn't have time to tend to it. |
The readme contains example of how to fetch
user.owned_events
, but this is recently deprecated and no longer returns event. This PR updates the docs to demo how to fetch events via organizations.