Skip to content

Conversation

@mnyon-grandkru
Copy link
Contributor

In your PR did you:

@rlgreen91
Copy link
Contributor

@mnyon-grandkru merge conflicts

@mnyon-grandkru mnyon-grandkru force-pushed the 4001_vendor_mimemagic branch 2 times, most recently from c1f9bfb to 1086d47 Compare March 28, 2021 14:42
Copy link
Contributor

@rlgreen91 rlgreen91 left a comment

Choose a reason for hiding this comment

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

I really just have the one question about the change to the acts_as_follower gem

Gemfile Outdated

# follower functionality
gem 'acts_as_follower', github: 'tcocca/acts_as_follower'
gem 'acts_as_follower'
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe the version listed on Rubygems has an issue with Rails 5+, which is why we switched to using the source here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it. Restoring github

@mnyon-grandkru mnyon-grandkru force-pushed the 4001_vendor_mimemagic branch from 6d5846f to 8ed607f Compare April 17, 2021 16:37
@rlgreen91 rlgreen91 merged commit 6330455 into main Apr 17, 2021
@rlgreen91 rlgreen91 deleted the 4001_vendor_mimemagic branch April 17, 2021 17:38
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.

3 participants