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

[TE-7348] Upgrade rails admin to latest version 3.2.0 #7

Merged
merged 419 commits into from
Nov 28, 2024

Conversation

tim-luong-goldenowl
Copy link
Collaborator

No description provided.

mshibuya and others added 30 commits December 18, 2021 14:30
…railsadminteam#3455)

* Upgrade Bootstrap to 5.1.3

Closes railsadminteam#3083

* Upgrade vendored Bootstrap as well

Also removed old theming functionality, since existing ones are not compatible with Bootstrap 5

* Migrate datetimepicker widget to flatpickr

* Reflect review and minor fix

- Fix inconsistent breakpoint
- Remove debug stuffs

* Apply form-control-sm to make datepicker field have the same size with other inputs

Co-authored-by: Alex Schmitt <[email protected]>

* Reflect review suggetions

- Prevent `true` showing up as a css class
- form-control-sm must come with form-control: https://getbootstrap.com/docs/5.1/forms/form-control/#sizing
- btn-small didn't have any effect
- Fix action menu having too narrow gap between icon and text

* Fix invalid closing tag </input>

* Update styles in Export and History action

* Fix spec issue, with removing the workaround

https://github.com/railsadminteam/rails_admin/runs/4747523252?check_suite_focus=true

* Add additional datetime format used somewhere in rails-i18n

* ProvidePlugin is no longer necessary

since all JS dependencies are ES module ready now

Co-authored-by: Alex Schmitt <[email protected]>
* Remove JavaScript library jquery-pjax

The library is no longer maintained and has not received a new release
since May 20, 2017. See https://github.com/defunkt/jquery-pjax

> Status of this project
>
> jquery-pjax is largely unmaintained at this point.

This also removes the Ruby gem rack-pjax which is also no longer
maintained and has not received a new release since Mar 24, 2019.

This is work towards the goal of dropping jQuery in favor of more
vanilla JavaScript solutions. As a dependency using jQuery, pjax was a
barrier toward this goal.

jquery-pjax was originally introduced in
645d678 (Nov 16, 2011). Presumably as a
page loading optimization, however it is not strictly necessary to
achieve the set of RailsAdmin features. After many years of improvements
in browser caching, loading, and JavaScript evaluation it is unclear if
this optimization is still necessary for a good UX.

Refs railsadminteam#2893
@jperdiguerra
Copy link

jperdiguerra commented Nov 28, 2024

Validation

Changes verified under https://kaligo.atlassian.net/browse/TE-7348

Impact: Only kaligo_web and hotel_genius
Screenshot 2024-11-28 at 11 56 18 AM

Reference in HG updated
https://github.com/Kaligo/hotel_genius/pull/161

@jperdiguerra jperdiguerra merged commit e79e3c3 into master Nov 28, 2024
2 checks passed
@jperdiguerra jperdiguerra deleted the feature/TE-7348 branch November 28, 2024 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.