Skip to content
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

Rails 5 support? #28

Open
aldefouw opened this issue Oct 11, 2017 · 20 comments
Open

Rails 5 support? #28

aldefouw opened this issue Oct 11, 2017 · 20 comments

Comments

@aldefouw
Copy link

aldefouw commented Oct 11, 2017

Has anyone made the gem work with Rails 5?

I did some preliminary testing on Rails 5, and there are some issues.

I made maily_herald-webui work with a few tweaks on a fork that I created:
https://github.com/aldefouw/maily_herald-webui/commits/master

My changes were mainly to some out of date dependencies. I also had to tweak the version code a bit to send the appropriate number of objects to the methods for Rails 5.

The web interface seems to be working pretty well, but I can't quite seem to get any mail to deliver. It seems like it gets tripped up when it tries to process the Mailers (I can post the error I am receiving if that helps) .. so I am wondering if Rails 5 changed something about ActionMailer that is incompatible.

I will do more research and try to fix it myself, but I just didn't want to be duplicating efforts if someone else is already working on Rails 5 compatibility.

If I can't make it work for Rails 5, I'll probably be forced to use some other solution. Looks really cool what's made so far!

Thanks!

@ljachymczyk
Copy link
Member

Actually, we're currently working on a new release that will support Rails 5 and introduce some new cool features, fixes and improvements. Will push some new commits in the next days. Keep watching our repo!

@aldefouw
Copy link
Author

@ljachymczyk - Awesome! Great to hear!

Are the updates going to be to Maily Herald WebUI as well?

@ljachymczyk
Copy link
Member

Both.

@ndbroadbent
Copy link

Awesome. I can't wait to use this in my Rails 5 app, it looks amazing! Please let me know if you need any beta testers :)

@ayb
Copy link

ayb commented Dec 1, 2017

@ljachymczyk How's the Rails 5 support going?

Is there a branch somewhere with the work in progress? Happy to try to help get it done, can you point me in the right direction?

@aldefouw
Copy link
Author

aldefouw commented Dec 1, 2017

@ayb - From what I can see, it looks like they're working on the 'ng' branch to me.

I've been knee deep in some other stuff right now otherwise I'd help too.

@ardahaal
Copy link
Contributor

ardahaal commented Dec 2, 2017

@ayb @aldefouw
Hey guys, thanks for the interest in this gem. That's correct - we're working on 'ng' branch on both core and webui gem. We've also created API (https://github.com/Sology/maily_herald-api) - feel free to check it out. It's not done yet due to other projects we're involved right now. Hopefully we can work with this soon (for now it's unlikely to finish this till the end of December).

@rgbskills
Copy link

Any news on rails 5 for WEB-UI ?

@Uysim
Copy link

Uysim commented Jun 25, 2018

How is it going with Rails 5?

1 similar comment
@thecryptovalley
Copy link

How is it going with Rails 5?

@Uysim
Copy link

Uysim commented Sep 8, 2018

I have my pull request here #30

@BlazPerus
Copy link

Rails 5?

@joshmn
Copy link

joshmn commented Oct 15, 2018

I have my pull request here #30

have you been using this in production? If so, how's it going?

@cpg
Copy link

cpg commented Nov 21, 2018

I get this error with that PR (well, with branch rails-5 in @Uysim's repo:

Multiple migrations have the name CreateMailyHeraldTables

even though there is exactly one migration with that name.

Update: It's picking one migration up from the gem at

~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/bundler/gems/maily_herald-af7243c8c099/db/migrate/20150205120443_create_maily_herald_tables.rb

Therefore, something is not right ... it should not be picked up.

Could you enable issues in your repo @Uysim, so that we follow up there? It looks like this repo is (semi?) abandoned.

@ljachymczyk
Copy link
Member

Hi @cpg,
This repo isn't abandoned. Rails 5 compatible release is on its way. You can already try it on 'ng' branch. From what I see @Uysim has copied most of that branch.

The issue with migrations you're experiencing comes from a fact that we've decided to keep Maily migrations inside the gem so you don't need to copy them to your application any more.
You just need to remove migration files from your application. There's a rake task that will help you do that: rake maily_herald:upgrade:migrations_fixup.
Let me know how that goes.

@Uysim
Copy link

Uysim commented Jan 2, 2019

@ljachymczyk I cloned from 'ng' branch. But I tested it. If my pull request is not helpful it is ok just ignore it

@kampeynco
Copy link

Any status on the release of Rails 5 compatibility?

@BlazPerus
Copy link

@ballotamerica Guess not :)

@ljachymczyk
Copy link
Member

Rails 5 and 6 are coming up in 'ng' branch. Feel free to try it out.

@ACPK
Copy link

ACPK commented May 30, 2019

@ljachymczyk - Are there any plans for the master branch to support Rails 5 and/or Rails 6. When I first found out about this gem, I left when I saw an open issue pertaining to Rails5. It wasn't until a few months later when I stumbled upon it again.

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

No branches or pull requests