Skip to content

Conversation

@enguerranws
Copy link
Collaborator

No description provided.

@enguerranws enguerranws changed the title Feautre / Download component Feature / Download component Feb 10, 2023
@garronej
Copy link
Collaborator

Super merci! @enguerranws

Comment on lines +151 to 153
"./File": "./dist/File.js",
"./Download": "./dist/Download.js",
"./Display": "./dist/Display.js",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's automatically generated, you don't need to worry about that.

import { fr } from "./fr";
import { cx } from "./tools/cx";
import { getLink } from "./link";
import { RegisteredLinkProps } from "./link";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Always import types as type

Comment on lines +7 to +8
import { getLink } from "./link";
import { RegisteredLinkProps } from "./link";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
import { getLink } from "./link";
import { RegisteredLinkProps } from "./link";
import { getLink, type RegisteredLinkProps } from "./link";

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, just forgot to review my own PR

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem!

Comment on lines +13 to +14
details: string;
label: string;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thoses two can be ReactNode

@garronej garronej merged commit 47446c7 into main Feb 10, 2023
@garronej garronej deleted the feature/download-component branch February 10, 2023 08:09
sbourdon13 pushed a commit to sbourdon13/react-dsfr that referenced this pull request Aug 11, 2023
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.

3 participants