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

Feature/add id to th component #1097

Merged

Conversation

GuillaumeKESTEMAN
Copy link
Contributor

Related issue

#1096

Description of the issue

  • Ajout de la propriété id dans le composant Th
  • Mise à jour de la story de Table afin de pouvoir y gérer les ids des différents headers

(+ mise à jour de deux snapshots pour que les tests puissent passer)

Person(s) for reviewing proposed changes

@samuel-gomez @MartinWeb

Important

Before creating a pull request run unit tests

$ npm test

# watch for changes
$ npm test -- --watch

# For a specific file (e.g., in packages/context/__tests__/command.test.js)
$ npm test -- --watch packages/table

@GuillaumeKESTEMAN GuillaumeKESTEMAN marked this pull request as draft December 22, 2023 11:21
@GuillaumeKESTEMAN GuillaumeKESTEMAN force-pushed the feature/add-id-to-th-component branch from 8e7eb91 to 1ec46d0 Compare December 22, 2023 14:10
@GuillaumeKESTEMAN GuillaumeKESTEMAN marked this pull request as ready for review December 22, 2023 14:11
@MartinWeb
Copy link
Contributor

j'aurais plutôt fait un seul commit pour cela. @samuel-gomez qu'en penses-tu ?

@samuel-gomez-axa
Copy link
Contributor

j'aurais plutôt fait un seul commit pour cela. @samuel-gomez qu'en penses-tu ?

oui on peut squash pour cette PR si ça te vas @GuillaumeKESTEMAN

@MartinWeb
Copy link
Contributor

j'aurais plutôt fait un seul commit pour cela. @samuel-gomez qu'en penses-tu ?

oui on peut squash pour cette PR si ça te vas @GuillaumeKESTEMAN

ok je squash 👍

@MartinWeb MartinWeb merged commit 722cd0c into AxaFrance:master Dec 27, 2023
@GuillaumeKESTEMAN GuillaumeKESTEMAN deleted the feature/add-id-to-th-component branch January 3, 2024 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants