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: #1246947 added Pagination component #3

Merged
merged 5 commits into from
Sep 6, 2023

Conversation

QuentinLeCaignec
Copy link
Contributor

Added Pagination component (+ test and story), added @storybook/preview-api dependency for story

@QuentinLeCaignec QuentinLeCaignec added the enhancement New feature or request label Aug 31, 2023
@QuentinLeCaignec QuentinLeCaignec self-assigned this Aug 31, 2023
@tonai
Copy link
Contributor

tonai commented Aug 31, 2023

Est-ce que pour le commit on ne mettrais pas le numéro du ticket dans le scope ?
C'est à dire avoir plutôt dans ton cas: feat (1246947): added Pagination component
De plus je me dis aussi qu'on pourrait avoir à terme des tickets dans Github, du coup je ne sais pas encore comment on va les gérer (est-ce qu'on utilise aussi Github pour gérer les tickets, est-ce qu'on synchronise redmine ?) mais peut-être il faudrait préciser que c'est un ticket redmine.
Du coup peut-être avoir comme convention: feat (redmine 1246947): added Pagination component ou en utilisant une version courte du genre feat (R1246947): added Pagination component ?

vapersmile
vapersmile previously approved these changes Aug 31, 2023
@tonai tonai mentioned this pull request Aug 31, 2023
@QuentinLeCaignec QuentinLeCaignec dismissed tonai’s stale review September 5, 2023 07:45

Je doit refuser à cause de la suggestion de code qui bloque

@QuentinLeCaignec QuentinLeCaignec merged commit 660896c into 0.1.x Sep 6, 2023
1 check passed
@QuentinLeCaignec QuentinLeCaignec deleted the feature/1246947_pagination branch September 18, 2023 14:15
tonai pushed a commit that referenced this pull request Sep 20, 2023
* feat(redmine 1246947): added Pagination component
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants