Replace render_anywhere gem with ActionController::Renderer to support Rails 6

[danielpuglisi]

Hi there,

Stumbled across this Gem yesterday and wanted to implement it in a Rails 6 app but had the same issue as #50 - so I created a quick PR.

Notes:

• I removed the ActionMailer::Base.default_url_options || {} in favour of using config/initializers/application_controller_renderer.rb. Checkout README update.
• When running the tests everything passes, but there are some errors coming from the ActiveJob logs. It seems as somehow the ActiveJob specs are using the Sidekiq spec methods of MyTexter#delayed_action – not sure how to fix this though.

[rafaltrojanowski]

@no-itsbackpack I'm using your fork with standard Rails 6 project (no-API) and it works without problems! Thanks for your work.

[dandlezzz]

I'm using this fork and it seems to work great. I did have to make one change, https://github.com/ondiem/textris/commit/38f0e2296867d28e17daa89ae26d0074733b3244.

[vdegove]

Hey there! I'd like to use Textris in a Rails 6 project. I liked a lot Textris in a Rails 5.2 project, I found it quite smart and easy to use to mirror mailer mechanisms into SMS.

Do you think it's safe to use this branch in a Rails 6 production environment? Or is there any major issue that prevents this branch from being merged into the main branch and for which you'd discourage to use it?

[danielpuglisi]

@vdegove I'm using it in production. I'm not sure if this gem is actively maintained though as no one of the core team responded to this PR so far.

[vdegove]

Allright, thanks for the feedback. This gem is way too practical, I'm just gonna fork your branch. I've seen that others have gone the same path even recently like here https://github.com/Semeia-io/textris… Anyway, may be just that folks at visuality.pl didn't even see the PR. @Marahin what do you think about this PR?

[Marahin]

Anyway, may be just that folks at visuality.pl didn't even see the PR. @Marahin what do you think about this PR?

I am no longer affiliated with the Open Source team at Visuality, but I will ping them on private channels to check it out.

I think the code looks good and I 👍 it.

[vdegove]

Hi @Marahin : have you talked to anybody at Visuality to restart maintaining this gem? It's a shame it's not maintained anymore. Or if it's not possible for them, would someone maintain an official fork and just change the README pointing towards this fork?

[rylanb]

Maybe https://github.com/visualitypl/textris/pull/62 fixes the failing tests via Sidekiq here?

Would love to get this merged if possible to support newer versions of Rails. I appreciate this gem and the effort that went into writing it!

[thubamamba]

Having problems with the gem in Rails 7 which this particular PR can resolve for so many users.