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

[bug] assure format is a symbol #27

Merged
merged 1 commit into from
Jan 8, 2020
Merged

[bug] assure format is a symbol #27

merged 1 commit into from
Jan 8, 2020

Conversation

pragmaticsc
Copy link
Contributor

I ran in to an issue when using Panoramic with Rails6 and Ruby 2.7.0. The error is caused by Mime[record.format] returning an object instead of to_sym. I have not investigated what version permutation introduced the change.

I've created an example project to reproduce the error: https://github.com/vandux/panoramic_error

Screen Shot 2019-12-28 at 8 51 00 AM

@andreapavoni andreapavoni merged commit c14036e into andreapavoni:master Jan 8, 2020
@maltesa
Copy link

maltesa commented Jun 27, 2020

Hi,

thank you for the great gem 👍
Could you make a new release including this fix and publish it on rubygems?

Best regards
Malte

@walidvb
Copy link

walidvb commented Nov 20, 2020

Spent quite a while before finding this PR :)
@andreapavoni, would be great if you could update the gem, or at least mention the issue in the README if something else is blocking you?

Thanks!

lucaguindani added a commit to catima/catima that referenced this pull request Jan 6, 2021
* remove_capistrano

* update_rails_ruby_gem

* update_rails_ruby_gem

* - namming fix
- remove not useful .keep files
- fix deprecation warning (`render template: XX`)

* fix error when rendering panoramic template (andreapavoni/panoramic#27)

* update panoramic gem to use a fork with a "fixed" master

* fix class naming

* fix assets loading with sprockets 4 (rails/sprockets-rails#328)

* Rollback changing case of HTML to Html. Fix zeitwerk error by adding HTML and SQL words into `ActiveSupport::Inflector` acronym.

* - active navbar item for custom menu too

* init BS update

* init BS update

* improve login form

* - rollback btn-sm to btn-xs and created dedicated class for this size of btn
- styling btn secondary

* nav dropdown on the right

* fix badge color

* form inline margin

* input with btn

* - fix form help + error (don't reduce size)

* - fix translated field "locale label" on input group append

* fix dropdown items

* fix clear btn on input search

* format blockquote

* fix navtabs

* media-heading doesn't exists on bs4

* fix h4 style

* fix badge color

* fix summernote editor

* fix all nav-tabs dom (and fix height)

* fix all nav-tabs dom (and fix height)

* cleanup bootstrap (remove BS from yarn in favor of gem version)

* cleanup bootstrap (remove BS from yarn in favor of gem version)

* fix thumbnails

* fix dropdown position

* smaller table padding

* fix popover

* fix summernote editor

* typo

* Fix form validation error on reference fields

* remove_capistrano

* try using last node version on travis cli

* change default style for active navbar item

* remove not used BootstrapForm override

* downgrade ruby version

* set version to all gemfile

* update to rails 6.1

* remove mini-racer gem + lock ruby gem version

* fix gemfile

* fixes for rails 6.1

* fixes for rails 6.1

* update font awesome rails

* bundle 2.1.4

* update gemfile with catima/panoramic

Co-authored-by: Manuel Wegria <abstract.s33d@gmail.com>
Co-authored-by: Luca Guindani <luca.guindani@gmail.com>
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.

4 participants