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

Bump kolibri-design-system from 5.0.0-rc3 to 5.0.0-rc5 #12651

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 11, 2024

Bumps kolibri-design-system from 5.0.0-rc3 to 5.0.0-rc5.

Release notes

Sourced from kolibri-design-system's releases.

v5.0.0-rc5

What's Changed

New components

Full Changelog: learningequality/kolibri-design-system@v5.0.0-rc4...v5.0.0-rc5

v5.0.0-rc4

What's Changed

New Components

KModal

KTooltip

KDS Initialization

Icons

KDS General Styling

Backported changes from v4.4.1

KCheckbox

  • 🚩BREAKING🚩 Removes internal state management for checked & indeterminate in KCheckbox.
    • Guidance: Now it is your responsibility to handle the change event and update whether or not the given checked and indeterminate props reflect the reality that you expect. By @​nucleogenesis in #744

New Contributors

Full Changelog: learningequality/kolibri-design-system@v5.0.0-rc3...v5.0.0-rc4

Changelog

Sourced from kolibri-design-system's changelog.

Changelog

Changelog is rather internal in nature. See release notes for the public overview and guidelines. Releases are recorded as git tags in the Github releases page.

Upcoming version 5.x.x (develop branch)

  • #776
    • Description: Bump KDS version to 5.0.0-rc5
    • Products impact: -.
    • Addresses: -.
    • Components: -.
    • Breaking: -.
    • Impacts a11y: -.
    • Guidance: -.

#776: learningequality/kolibri-design-system#776

  • #727
    • Description: Initial implementation of KTable component
    • Products impact: New Component
    • Addresses: #328
    • Components: KTable
    • Breaking: No
    • Impacts a11y: Yes
    • Guidance:

#727: learningequality/kolibri-design-system#727

  • #766
    • Description: Bump KDS version to 5.0.0-rc4
    • Products impact: -.
    • Addresses: -.
    • Components: -.
    • Breaking: -.
    • Impacts a11y: -.
    • Guidance: -.

#766: learningequality/kolibri-design-system#766

  • #765
    • Description: Fixes update changelog workflow to avoid expecting a comment after the invisible comment <!-- [DO NOT REMOVE-USED BY GH ACTION] CHANGELOG START -->.
    • Products impact: none.
    • Addresses: -
    • Components: -
    • Breaking: -
    • Impacts a11y: -

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added javascript Pull requests that update Javascript code TAG: dependencies Pull requests that update a dependency file labels Sep 11, 2024
@MisRob
Copy link
Member

MisRob commented Sep 12, 2024

Regarding KModal breaking change, these are the two places in Kolibri where appendToRoot needs to be renamed to appendToOverlay. Then it should work just fine.

https://github.com/search?q=repo%3Alearningequality%2Fkolibri%20appendToRoot&type=code

@nucleogenesis
Copy link
Member

cherry-picked 4f28350 here to get tests passing

@github-actions github-actions bot added the APP: Learn Re: Learn App (content, quizzes, lessons, etc.) label Sep 24, 2024
@MisRob
Copy link
Member

MisRob commented Sep 24, 2024

Thanks a lot @nucleogenesis, code changes make sense to me. I think after QA you mentioned earlier, we should be ready for merge. Thanks for jumping in to support this.

@pcenov
Copy link
Member

pcenov commented Sep 25, 2024

Hi @nucleogenesis - no issues observed while testing the selection on the user table, everything is working as expected.

dependabot bot and others added 3 commits September 25, 2024 13:08
Bumps [kolibri-design-system](https://github.com/learningequality/kolibri-design-system) from 5.0.0-rc3 to 5.0.0-rc5.
- [Release notes](https://github.com/learningequality/kolibri-design-system/releases)
- [Changelog](https://github.com/learningequality/kolibri-design-system/blob/develop/CHANGELOG.md)
- [Commits](learningequality/kolibri-design-system@v5.0.0-rc3...v5.0.0-rc5)

---
updated-dependencies:
- dependency-name: kolibri-design-system
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
…kbox

KCheckbox no longer maintains its own internal state, rather, it is just
like a fancy button which emits an event that it's been clicked and the
parent component must react and update the value it passes to `checked`
or `indeterminate` accordingly.

These tests rely on KCheckbox to emit its internal state whenever it
emits `change` and that no longer is the case given KCheckbox doesn't
keep an internal state.

I've tweaked some of the conditionals in UserTable that relied on that
emitted `checked` value so that they instead get the same information
from existing available values.
@nucleogenesis nucleogenesis force-pushed the dependabot/npm_and_yarn/kolibri-design-system-5.0.0-rc5 branch from 3f36ea9 to d5ddd80 Compare September 25, 2024 20:08
@nucleogenesis nucleogenesis merged commit 67b4c75 into develop Sep 25, 2024
37 checks passed
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/kolibri-design-system-5.0.0-rc5 branch September 25, 2024 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Learn Re: Learn App (content, quizzes, lessons, etc.) DEV: frontend javascript Pull requests that update Javascript code TAG: dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants