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: LEAP-1351: LSF customization for bulk annotation #6203

Open
wants to merge 25 commits into
base: develop
Choose a base branch
from

Conversation

Gondragos
Copy link
Collaborator

@Gondragos Gondragos commented Aug 14, 2024

This PR adds

  • "annotation:bulk" interface that allows to hide all panels and elements that can't be set equally to group of annotaions
  • small fixes for custom controll buttons in bottom bar to be able to use them
  • fix the way we deliver config in cypress tests

PR fulfills these requirements

  • Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
  • Tests for the changes have been added/updated (for bug fixes/features)
  • Docs have been added/updated (for bug fixes/features)
  • Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
  • Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)

  • Product design
  • Backend (Database)
  • Backend (API)
  • Frontend

What feature flags were used to cover this change?

fflag_feat_all_leap_1181_bulk_annotation_short

Does this PR introduce a breaking change?

  • Yes, and covered entirely by feature flag(s)
  • Yes, and covered partially by feature flag(s)
  • No
  • Not sure (briefly explain the situation below)

What level of testing was included in the change?

(check all that apply)

  • e2e
  • integration
  • unit

Which logical domain(s) does this change affect?

Editor, BottomBar

- hide dependant controls
- hide panels
- for bulk annotation mode displaying
- for custom controls rendering and modify on fly
- fix delivering the config from the cypress environment into the test environment
@github-actions github-actions bot added the feat label Aug 14, 2024
Copy link

netlify bot commented Aug 14, 2024

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit bd5b0ed
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/6740aef9336a6b0009d9535f

Copy link

netlify bot commented Aug 14, 2024

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit bd5b0ed
🔍 Latest deploy log https://app.netlify.com/sites/heartex-docs/deploys/6740aef91d64800008b0330f

@hlomzik
Copy link
Collaborator

hlomzik commented Aug 14, 2024

Would not it be better to name interface "annotation:bulk"? That will keep it in a shape of namespace:detail like for lot of other interfaces. I know they are a mess but that's a really lowcost change on an early stage with small benefit

Separated bulk mode and independence checks into variables for readability. This makes the conditional logic clearer and potentially reduces redundancy in code review and debugging processes. No functional changes have been introduced.
# Conflicts:
#	web/libs/editor/src/components/BottomBar/BottomBar.jsx
@Gondragos
Copy link
Collaborator Author

Gondragos commented Aug 26, 2024

/git merge develop

Workflow run
Successfully merged: Already up to date.

@Gondragos
Copy link
Collaborator Author

Gondragos commented Aug 29, 2024

/git merge develop

Workflow run
Successfully merged: 387 files changed, 6483 insertions(+), 9039 deletions(-)

@Gondragos
Copy link
Collaborator Author

Gondragos commented Sep 2, 2024

/git merge develop

Workflow run
Successfully merged: 29 files changed, 94 insertions(+), 69 deletions(-)

@robot-ci-heartex
Copy link
Collaborator

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@robot-ci-heartex
Copy link
Collaborator

This PR was closed because it has been stalled for 10 days with no activity.

# Conflicts:
#	web/libs/editor/src/components/App/App.jsx
#	web/libs/editor/src/components/BottomBar/Controls.tsx
#	web/libs/editor/src/stores/AppStore.js
#	web/libs/editor/src/tags/control/ClassificationBase.js
#	web/libs/editor/src/tags/visual/Collapse.jsx
#	web/libs/editor/src/tags/visual/View.jsx
@Gondragos
Copy link
Collaborator Author

Gondragos commented Nov 22, 2024

/git merge develop

Workflow run
Successfully merged: 29 files changed, 78 insertions(+), 117 deletions(-)

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.

4 participants