Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improvement to Annotator management screen (resolves #190) #232

Merged
merged 19 commits into from
Oct 5, 2022

Conversation

davidwilby
Copy link
Contributor

@davidwilby davidwilby commented Aug 25, 2022

  • Project annotators list changed to a sortable table so you can sort their stats
    • sort by status: undertaking training/test, completed training/test awaiting approval, annotating, completed annotation, rejected
    • sort by who is waiting - dedicated sortable column showing who is waiting to be approved
  • Add column indicating project membership - I'm really not sure what I meant by this
  • Improve clarity on annotator management screen showing whether annotator has completed training/approval/score etc.
    • Especially for users waiting for approval, currently warning icon clashes with the yellow background
  • Change tooltip for on Complete and Reject buttons, make it more clear
  • Make sure annotators are removed from list when rejected/complete - I'm not sure this is required
  • Hide the add annotators into separate popup window
  • Perform batch actions like making multiple users annotators
  • Don't show 'make active' if user has met annotation quota
  • Handle test score (show as red for instance)
  • testing (use test fixture)

resolves #190

@davidwilby davidwilby requested a review from twinkarma August 25, 2022 17:23
@github-actions
Copy link

github-actions bot commented Aug 25, 2022

Jest Coverage

File % Stmts % Branch % Funcs % Lines Uncovered Line #s
All files 80.11 79.16 78.26 80.11
File % Stmts % Branch % Funcs % Lines Uncovered Line #s
All files 80.11 79.16 78.26 80.11
_src/components 74.73 75.51 79.16 74.73
_ AnnotationRenderer.vue 73.62 75.51 76.19 73.62 114,123-124,178,201,221,228-242,250,256-266,283
_ MarkdownEditor.vue 100 100 100 100
_ MarkdownRenderer.vue 100 100 100 100
_src/components/annotation 78.57 86.66 76.47 78.57
_ CheckboxInput.vue 85.71 100 100 85.71 19
_ HtmlDisplay.vue 80 100 100 80 26-30
_ InputErrorDisplay.vue 100 0 100 100 19-25
_ RadioInput.vue 71.42 100 66.66 71.42 19,26
_ SelectorInput.vue 71.42 100 66.66 71.42 20,27
_ TextInput.vue 75 100 66.66 75 26
_ TextareaInput.vue 66.66 100 50 66.66 20
_src/enum 100 100 100 100
_ DocumentTypes.js 100 100 100 100
_src/jrpc 90.32 83.33 75 90.32
_ index.js 90.32 83.33 75 90.32 29,38-39
_src/utils 100 100 100 100
_ annotations.js 100 100 100 100
_tests/unit 100 100 100 100
_ globalVue.js 100 100 100 100

@davidwilby davidwilby changed the title Annotator management screen (resolves #190) Improvement to Annotator management screen (resolves #190) Aug 31, 2022
@davidwilby davidwilby merged commit b9cbe6a into dev Oct 5, 2022
@davidwilby davidwilby deleted the annotator_mgmt branch October 5, 2022 12:34
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant