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

feat: add exports for experimental components #464

Merged
merged 4 commits into from
Sep 16, 2024

Conversation

martimalek
Copy link
Contributor

@martimalek martimalek commented Aug 27, 2024

What

Adds exports for the experimental folder, so that when distributing the package it can be accessed directly.

Media

Now the experimental components can be accessed like:

import { Button, Chip } from '@freenow/wave/experimental'

Why

To allow to directly access experimental features, without needing to know the internal file structure of the package.

How

  • Adds exports field in package.json to define the default + the experimental exports
  • Adds typesVersions field to package.json to override the experimental types (so they don't come from the main index.d.ts file)

Extra

I'm opening this as a draft PR because the experimental components are not ready to be distributed yet, once we have a first experimental component we can merge this.

I've already tested this approach in several rc releases, if you want to try it out as well you can check 2.6.0-rc.8. I've rolled back the version in package.json so that it gets automatically updated by semantic-release once the PR is merged.

The linting issues that you see are due to comments in some experimental components, those should be solved as part of the PRs for each component.

@martimalek martimalek added enhancement New feature or request CI/CD Efforts that will impact our integration/deployment pipeline labels Aug 27, 2024
@martimalek martimalek self-assigned this Aug 27, 2024
package.json Outdated Show resolved Hide resolved
@alatielle alatielle merged commit c71ddf3 into poc-experimental Sep 16, 2024
8 of 10 checks passed
@alatielle alatielle deleted the experimental-publishing branch September 16, 2024 08:57
alatielle added a commit that referenced this pull request Sep 19, 2024
* feat(experimental): add Chip and Text

Add experimental theme and utils.

* feat(experimental): add Button WIP

* chore(storybook): upgrade to latest v8

* fix(raddi): fix order

* feat(font): load Roboto Flex in Storybook

* style(chip): remove commented/unused code

* feat(button): adjust style

* feat(text): add story

* feat(button): add secondary

* feat(storybook): re-order experimental folder

* feat(chip): remove react import from story

* feat(text): add kitchen sink

* feat(input): add input

* feat(input): export and fix story

* Disp 105/css colors (#466)

* feat: add experimental colors

* feat(button): use color variables

* feat(storybook): add experimental theme

* chore: update package-lock

---------

Co-authored-by: Lena Rashkovan <lena.rashkovan@free-now.com>

* Disp 140/button (#467)

* feat(storybook): add experimental themes for rider and driver

* feat(button): add styling

* feat(button): add pressed and focus states

* fix(text): fix typography

* feat(experimental): use rems in spaces

---------

Co-authored-by: Lena Rashkovan <lena.rashkovan@free-now.com>

* feat: add exports for experimental components (#464)

* feat: add exports for experimental components

* chore: rollback manual rc version

* feat(experimental): add color palette and global styles to export

* chore: put codemods back

---------

Co-authored-by: Lena Rashkovan <lena.rashkovan@free-now.com>

* Disp 139/input (#468)

* feat(input): replace base input with react-aria component

* feat(text-field): add basic styles and structure

* feat(text-field): add a flying label

* feat(text-field): add a flying label

* feat(text-field): add clearing

* chore(text-field): silence the style linter

* feat(text-field): add auto grow

* fix(text-field): fix overflow jump

---------

Co-authored-by: Lena Rashkovan <lena.rashkovan@free-now.com>

---------

Co-authored-by: Artur Miglio <artur.miglio@free-now.com>
Co-authored-by: Lena Rashkovan <lena.rashkovan@free-now.com>
Co-authored-by: martimalek <46452321+martimalek@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Efforts that will impact our integration/deployment pipeline enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

4 participants