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

Fixes #21253 - fix latest events widget #4903

Merged
merged 1 commit into from Oct 12, 2017
Merged

Fixes #21253 - fix latest events widget #4903

merged 1 commit into from Oct 12, 2017

Conversation

akofink
Copy link
Contributor

@akofink akofink commented Oct 9, 2017

No description provided.

@theforeman-bot
Copy link
Member

Issues: #21253

Copy link
Member

@dLobatog dLobatog left a comment

Choose a reason for hiding this comment

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

@tbrisker Could you check this out? I'm not quite sure by looking at the git history why this was so complicated before. Thanks!

@ares
Copy link
Member

ares commented Oct 11, 2017

Looking at ee00829 that introduced it, the workaround was required because of older rails bug. @tbrisker covered it with tests which are green so I believe this is no longer necessary. I tested this also on older stack and it fixes the last event widget. Latest errata is still broken but that most likely needs to be fixed in katello, @akofink do you plan to look at that one too? The linked BZ mentions both widgets so after we merge this, it would be marked as fixed...

Tested on

  • foreman 1.15.4
  • katello 3.4.5
  • rails 4.2.6

If noone objects, I'll merge this.

@akofink
Copy link
Contributor Author

akofink commented Oct 11, 2017

Katello/katello#6995 and http://projects.theforeman.org/issues/21254 are for the errata widget. I just added 21254 to the BZ. Feel free to merge! Thanks!

@timogoebel timogoebel merged commit fa90853 into theforeman:develop Oct 12, 2017
@timogoebel
Copy link
Member

Merged, thanks @akofink, @ares, @dLobatog.

@akofink akofink deleted the 21253 branch October 12, 2017 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants