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

Merge original back into our codebase #1

Open
wants to merge 371 commits into
base: master
Choose a base branch
from

Conversation

lopesrb
Copy link
Member

@lopesrb lopesrb commented Apr 29, 2021

No description provided.

gmq and others added 30 commits November 20, 2020 17:13
I noticed when I tried this that the import, it wasnt working properly, swapping to the above import worked (had to inspect the node_modules folder.)
Fix import of SCSS in webpacker.
difernandez and others added 30 commits June 9, 2023 15:39
…m-auto-deploy

Release beta 2 and configure npm auto deploy
Fix link to PR #451 in CHANGELOG.md
We wont't require ruby 3 just yet, but for testing purposes we need to remove it
fix(tests): fix chromedriver error for capybara tests
Mainly, the 'contains' predicate, that was used as default in some components. These predicates were removed from activeadmin v3, since they were backports. More info here activeadmin/activeadmin#8010
Fix typo in slim-select_default.md
The following deprecation warning was addressed.

```
Deprecation Warning: Sass's behavior for declarations that appear after nested
rules will be changing to match the behavior specified by CSS in an upcoming
version. To keep the existing behavior, move the declaration above the nested
rule. To opt into the new behavior, wrap the declaration in `& {}`.

More info: https://sass-lang.com/d/mixed-decls

    ╷
5   │ ┌     label {
6   │ │       display: none;
7   │ │     }
    │ └─── nested rule
... │
5   │ ┌     label {
6   │ │       display: none;
7   │ │     }
    │ └─── nested rule
... │
9   │       margin-bottom: 5px;
    │       ^^^^^^^^^^^^^^^^^^ declaration
    ╵
    vendor/bundle/gems/activeadmin_addons-1.10.1/app/assets/stylesheets/activeadmin_addons/inputs/date-ti
me-picker-filter.scss 9:5  @import
(snip)
```

This warning is due to the following change in dart-sass.
sass/dart-sass#2267
…on-warning

Suppress sass deprecation warning
Add confirm_message to toggle-bool-switch
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.