Skip to content

[13] Add checkbox cells to WorkpackageArtefactPageDescription#29

Merged
lfasani merged 1 commit intomainfrom
nco/enh/checkbox
Mar 31, 2026
Merged

[13] Add checkbox cells to WorkpackageArtefactPageDescription#29
lfasani merged 1 commit intomainfrom
nco/enh/checkbox

Conversation

@ncouvert
Copy link
Copy Markdown
Collaborator

@ncouvert ncouvert commented Mar 18, 2026 •
edited
Loading

[13] Add checkbox cells to WorkpackageArtefactPageDescription instead of the old text field

Issue: #13

@ncouvert ncouvert force-pushed the nco/enh/checkbox branch 4 times, most recently from 140b712 to 1054b8f Compare March 18, 2026 14:06
@ncouvert ncouvert mentioned this pull request Mar 20, 2026
Closed
lfasani
lfasani requested changes Mar 27, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@lfasani lfasani left a comment

Choose a reason for hiding this comment

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

Commit message to change to

[13] Add checkbox cells to WorkpackageArtefactPageDescription instead of the old text field

Issue: #13

Why are you talking about the cherry pick that corresponds to the same PR?
Do not talk about Bug: eclipse-sirius/sirius-web#6141. Instead, put a message with a reference to the pepper issue 13 and this SW issue in the other commit that switches to the SW version that allows the fix

@ncouvert
Copy link
Copy Markdown
Collaborator Author

ncouvert commented Mar 27, 2026

Done
I'v mentionned the bug in this commit : 7d0beda

Commit message to change to

[13] Add checkbox cells to WorkpackageArtefactPageDescription instead of the old text field

Issue: #13

Why are you talking about the cherry pick that corresponds to the same PR? Do not talk about Bug: eclipse-sirius/sirius-web#6141. Instead, put a message with a reference to the pepper issue 13 and this SW issue in the other commit that switches to the SW version that allows the fix

lfasani
lfasani requested changes Mar 27, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@lfasani lfasani left a comment

Choose a reason for hiding this comment

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

update the commit message

@ncouvert ncouvert force-pushed the nco/releng/bump branch 2 times, most recently from 4f9c130 to c706d0a Compare March 27, 2026 14:06
@ncouvert ncouvert force-pushed the nco/enh/checkbox branch 2 times, most recently from 90889f7 to c5e76b6 Compare March 27, 2026 14:39
Base automatically changed from nco/releng/bump to main March 30, 2026 07:49
@lfasani lfasani merged commit c0fae92 into main Mar 31, 2026
3 checks passed
@lfasani lfasani deleted the nco/enh/checkbox branch March 31, 2026 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lfasani lfasani lfasani approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ncouvert @lfasani