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: lock board #3955

Merged
merged 66 commits into from
Jul 19, 2024
Merged

feat: lock board #3955

merged 66 commits into from
Jul 19, 2024

Conversation

brandstetterm
Copy link
Collaborator

@brandstetterm brandstetterm commented Mar 8, 2024

Description

Frontend changes related to the locking of a board. If a board is locked, participants (without moderation permissions) are no longer able to create, update and delete notes, dispatch board reactions, add or remove reactions to a note, add or remove votes to a note. Therefore, there have been some changes in the frontend (e.g. gray out buttons, hide buttons, etc.). A detailed list of the changes can be found below.

Closes #3538

Changelog

  • NoteInput: Disable textarea, add lock icon, gray out plus icon
  • StackView: Disable textarea to edit note content, hide delete and unstack buttons
  • Action Bars: Disable button to dispatch board reactions, automatically close board reactions menu if board gets locked
  • Notes: Hide button to open reactions dropdown, disable button to remove note reactions, hide button to add a vote and disable button to remove a vote
  • Update several tests and snapshots

Future tasks

  • Implement notifications manager and dispatch notification that board is locked on join

Visual changes

Screenshot 2024-03-18 at 17 31 49
Screenshot 2024-03-18 at 17 32 05

@brandstetterm brandstetterm self-assigned this Mar 8, 2024
Copy link

octomind-dev bot commented Mar 11, 2024

🐙 Octomind

Test Report: 10/13 successful.

description status details
change avatar Passed ✅ click
check Privacy Policy Passed ✅ click
check terms & conditions Passed ✅ click
close cookie banner - front page Passed ✅ click
close cookie banner - sign-in Passed ✅ click
create and delete board columns Failed ❌ click
create_and_delete_notes_and_actions_v2 Failed ❌ click
create lean coffee board Passed ✅ click
edit_notes_and_actions_v5 Failed ❌ click
get started Passed ✅ click
share_session Passed ✅ click
sign-in Passed ✅ click
test all ways to open the setup flow Passed ✅ click

commit sha: 2fffed9

Copy link

octomind-dev bot commented Mar 11, 2024

🐙 Octomind

Test Report: 12/13 successful.

description status details
change avatar Passed ✅ click
check Privacy Policy Passed ✅ click
check terms & conditions Passed ✅ click
close cookie banner - front page Passed ✅ click
close cookie banner - sign-in Passed ✅ click
create and delete board columns Passed ✅ click
create_and_delete_notes_and_actions_v2 Passed ✅ click
create lean coffee board Passed ✅ click
edit_notes_and_actions_v5 Failed ❌ click
get started Passed ✅ click
share_session Passed ✅ click
sign-in Passed ✅ click
test all ways to open the setup flow Passed ✅ click

commit sha: 7af6b86

Copy link

octomind-dev bot commented Mar 12, 2024

🐙 Octomind

Test Report: 11/13 successful.

description status details
change avatar Passed ✅ click
check Privacy Policy Passed ✅ click
check terms & conditions Passed ✅ click
close cookie banner - front page Passed ✅ click
close cookie banner - sign-in Passed ✅ click
create and delete board columns Passed ✅ click
create_and_delete_notes_and_actions_v2 Passed ✅ click
create lean coffee board Failed ❌ click
edit_notes_and_actions_v5 Failed ❌ click
get started Passed ✅ click
share_session Passed ✅ click
sign-in Passed ✅ click
test all ways to open the setup flow Passed ✅ click

commit sha: 7b6091e

Copy link

octomind-dev bot commented Mar 12, 2024

🐙 Octomind

Test Report: 11/13 successful.

description status details
change avatar Passed ✅ click
check Privacy Policy Passed ✅ click
check terms & conditions Passed ✅ click
close cookie banner - front page Passed ✅ click
close cookie banner - sign-in Passed ✅ click
create and delete board columns Passed ✅ click
create_and_delete_notes_and_actions_v2 Passed ✅ click
create lean coffee board Passed ✅ click
edit_notes_and_actions_v5 Failed ❌ click
get started Passed ✅ click
share_session Failed ❌ click
sign-in Passed ✅ click
test all ways to open the setup flow Passed ✅ click

commit sha: 315cf92

Copy link

octomind-dev bot commented Mar 12, 2024

🐙 Octomind

Test Report: 12/13 successful.

description status details
change avatar Passed ✅ click
check Privacy Policy Passed ✅ click
check terms & conditions Passed ✅ click
close cookie banner - front page Passed ✅ click
close cookie banner - sign-in Passed ✅ click
create and delete board columns Passed ✅ click
create_and_delete_notes_and_actions_v2 Passed ✅ click
create lean coffee board Passed ✅ click
edit_notes_and_actions_v5 Failed ❌ click
get started Passed ✅ click
share_session Passed ✅ click
sign-in Passed ✅ click
test all ways to open the setup flow Passed ✅ click

commit sha: a18c165

Copy link

octomind-dev bot commented May 29, 2024

🐙 Octomind

Test Report: 0/14 successful.

description status details
About Section Visibility Test Failed ❌ click
change avatar Failed ❌ click
check Privacy Policy Failed ❌ click
check terms & conditions Failed ❌ click
close cookie banner - front page Failed ❌ click
close cookie banner - sign-in Failed ❌ click
create and delete board columns Failed ❌ click
create_and_delete_notes_and_actions_v2 Failed ❌ click
create lean coffee board Failed ❌ click
edit_notes_and_actions_v5 Failed ❌ click
get started Failed ❌ click
share_session Failed ❌ click
sign-in Failed ❌ click
test all ways to open the setup flow Failed ❌ click

commit sha: f353a3d

This comment has been minimized.

@Schwehn42 Schwehn42 added the Review Needed This pull request is ready for review label May 29, 2024
@Schwehn42
Copy link
Collaborator

added feats: #4144 and #4145, ready to review now

# Conflicts:
#	src/components/MenuBars/MenuBars.tsx
#	src/components/Note/NoteReactionList/NoteReactionList.tsx
#	src/components/NoteInput/NoteInput.scss
#	src/components/NoteInput/NoteInput.tsx
#	src/components/TooltipButton/TooltipButton.scss

This comment has been minimized.

This comment has been minimized.

# Conflicts:
#	server/api.postman_collection.json
#	src/components/Note/NoteReactionList/NoteReactionChip/NoteReactionChip.tsx
#	src/components/Note/NoteReactionList/NoteReactionList.tsx
#	src/components/NoteInput/NoteInput.scss

This comment has been minimized.

@BenedictHomuth
Copy link
Collaborator

Should the feature of locking a board also be an option, when creating a board?
Meaning that it should be possible to set the "isLocked" field in the API request when creating a board?

If so I would add it. I do not remember if this was requested or not when we formulated this feature.

@Schwehn42
Copy link
Collaborator

@BenedictHomuth

Should the feature of locking a board also be an option, when creating a board?

This isn't an AC in #3538, so I would hold on implementing it, since the create board screen will be overhauled soon anyways.

Copy link
Collaborator

@laila-rin laila-rin left a comment

Choose a reason for hiding this comment

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

looks good to me, works well.

Would it make sense to have a little lock icon for the moderator also, so they know if the board is locked or not? Atm it's just the non-moderator participants that have a lock icon to indicate that, right?

@Schwehn42
Copy link
Collaborator

@laila-rin Good point, I actually like the initial idea from the issue (#3538) where instead of toggle in board settings, it would be an button on the right where the rest of the moderator tools (start vote, start timer ...) are. That way, moderators could always see whether it's active, and also toggle it with one click instead of two.

What do you think about this? Also, does it make more sense to just use this version and make these changes in a new PR, if @SelinaBuff approves of the idea?

# Conflicts:
#	src/components/NoteDialogComponents/NoteDialogNoteComponents/NoteDialogNoteContent.tsx
Copy link

The deployment to the dev cluster was successful. You can find the deployment here: https://3955.development.scrumlr.fra.ics.inovex.io
This deployment is only for testing purposes and will be deleted after 1 week.
To redeploy rerun the workflow.
DO NOT STORE IMPORTANT DATA ON THIS DEPLOYMENT

Deployed Images
  • ghcr.io/inovex/scrumlr.io/scrumlr-frontend:sha-d789bd8

  • ghcr.io/inovex/scrumlr.io/scrumlr-server:sha-d789bd8

Copy link

octomind-dev bot commented Jul 19, 2024

🐙 Octomind

Test Report: 13/14 successful.

description status details
About Section Visibility Test Passed ✅ click
change avatar Passed ✅ click
check Privacy Policy Passed ✅ click
check terms & conditions Passed ✅ click
close cookie banner - front page Passed ✅ click
close cookie banner - sign-in Passed ✅ click
create and delete board columns Passed ✅ click
create_and_delete_notes_and_actions_v2 Failed ❌ click
create lean coffee board Passed ✅ click
edit_notes_and_actions_v5 Passed ✅ click
get started Passed ✅ click
share_session Passed ✅ click
sign-in Passed ✅ click
test all ways to open the setup flow Passed ✅ click

commit sha: d789bd8

@Schwehn42 Schwehn42 removed the Review Needed This pull request is ready for review label Jul 19, 2024
@Schwehn42 Schwehn42 added this pull request to the merge queue Jul 19, 2024
Merged via the queue into main with commit 8bebd00 Jul 19, 2024
10 checks passed
@Schwehn42 Schwehn42 deleted the mb/lock-board branch July 19, 2024 15:05
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.

Add option to lock board
4 participants