Feature: ActiveJob Integration and Documentation

[codebycliff]

Description

This pull request makes it so that decorators implement GlobalID so they can be passed to background jobs. While you can now pass a decorated object to a background job, it won't be decorated when it is deserialized. This may lead to some confusion, so it is documented in the README.

Testing

Try to pass a decorated object to a background job and it should no longer throw ActiveJob::SerializationError.

To-Dos

• tests
• documentation

References

• GitHub Issue 663
• GitHub Pull Request #669 containing initial documentation

[codebycliff]

@syguer I don't know if you are still interested in helping, but if you are, I'd appreciate a look at this when you get a chance.

[syguer]

sure 😄
Please give me some time, I'll look.

[syguer]

@codebycliff Sorry for late. Looks good! 😄
In addition, It might be good to add some spec for to_global_id like this

let(:post) { Post.create }
let(:decorated_post) { post.decorate }
subject { decorated_post.to_global_id }

it { is_expected.to eq post.to_global_id }

This might be exaggerate spec. I respect your decision 👍

[syguer]

Thanks!
LGTM 👍