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.

Sign up for GitHub

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

Jump to bottom

feat: duplicate regular board #1281

Merged
merged 23 commits into from
Mar 23, 2023
Merged

feat: duplicate regular board #1281

merged 23 commits into from
Mar 23, 2023

Conversation

jpvsalvador
Copy link
Contributor

Relates to:

Fixes:

Screenshots

image

image

Proposed Changes

  • Allow a user with permissions to duplicate a regular board
  • The duplicated board has to have a different name but keeps all the users, responsibles, cards, votes and comments

This pull request closes #1250

@jpvsalvador jpvsalvador self-assigned this Mar 21, 2023
@jpvsalvador jpvsalvador linked an issue Mar 21, 2023 that may be closed by this pull request
[FEATURE]: duplicate regular board #1250
Closed
@jpvsalvador jpvsalvador marked this pull request as draft March 21, 2023 12:57
@jpvsalvador jpvsalvador marked this pull request as ready for review March 22, 2023 16:18
nunocaseiro
nunocaseiro requested changes Mar 23, 2023
View reviewed changes
backend/src/modules/boards/applications/duplicate-board.use-case.ts Outdated Show resolved Hide resolved
backend/src/modules/boards/applications/duplicate-board.use-case.spec.ts Outdated Show resolved Hide resolved
backend/src/modules/boards/applications/duplicate-board.use-case.spec.ts Outdated Show resolved Hide resolved
backend/src/modules/boards/applications/duplicate-board.use-case.ts Outdated Show resolved Hide resolved
backend/src/modules/boards/applications/duplicate-board.use-case.ts Outdated Show resolved Hide resolved
backend/src/modules/boards/controller/boards.controller.spec.ts Outdated Show resolved Hide resolved
backend/src/modules/boards/controller/boards.controller.ts Outdated Show resolved Hide resolved
frontend/src/components/CardBoard/CardBody/CardEnd.tsx Outdated Show resolved Hide resolved
frontend/src/components/CardBoard/CardBody/CardEnd.tsx Outdated Show resolved Hide resolved
frontend/src/hooks/useBoard.tsx Show resolved Hide resolved
nunocaseiro
nunocaseiro approved these changes Mar 23, 2023
View reviewed changes
Copy link
Contributor

@nunocaseiro nunocaseiro left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@jpvsalvador jpvsalvador merged commit 70c3cf2 into main Mar 23, 2023
@jpvsalvador jpvsalvador deleted the 1250-feature-duplicate-regular-board branch March 23, 2023 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nunocaseiro nunocaseiro nunocaseiro approved these changes

Assignees

@jpvsalvador jpvsalvador

Labels
backend frontend
Projects
SPLIT • Backend
Status: Done
SPLIT • Frontend
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEATURE]: duplicate regular board
2 participants
@jpvsalvador @nunocaseiro