Skip to content
This repository has been archived by the owner on Feb 20, 2024. It is now read-only.

Change text for step away reasons #163

Closed
konradlang opened this issue Dec 16, 2022 · 0 comments · Fixed by #181
Closed

Change text for step away reasons #163

konradlang opened this issue Dec 16, 2022 · 0 comments · Fixed by #181

Comments

@konradlang
Copy link
Contributor

konradlang commented Dec 16, 2022 •
edited
Loading

Change the text for available options when step away is chosen in workflow.
Once a Biobank makes the decision to ‘Step Away’ they are able to give reasons. Currently these are ‘Reason 1’ and ‘Reason 2’.

Recommendations for Reasons to Step away:

Code Reason Rationale
reason1 Samples/data not available This is an option for biobanks who are interested and I think should be given here. At present I don’t see any reason why a biobank would know that this is an option after they chose the alternative ‘interested checking availability’. This seems like the most likely reason someone would ‘step away’
reason2 Samples/data available but not accessible for given purpose This is an option for biobanks who are interested and I think should be given here. At present I don’t see any reason why a biobank would know that this is an option after they chose the alternative ‘interested checking availability’. This seems like the second most likely reason someone would ‘step away’
reason3 No response from biobank This is for Network managers who may not be able to get a response from the biobank/sample custodian
reason4 Not able to attend query at this time This is for biobanks who may have the requested material however are not able to check for whatever reason
konradlang added a commit that referenced this issue Jan 26, 2023
@konradlang
changed text for step away reasons - #163 (#163)
32b1e76
@konradlang konradlang linked a pull request Feb 3, 2023 that will close this issue
fix: wording for stepaway texts on UI #181
Merged
RadovanTomik added a commit that referenced this issue Feb 7, 2023
@konradlang @RadovanTomik
fix: wording for stepaway texts on UI (#181)
159865f 
* Throttling the requests by waiting for x seconds in the range [0,3] before requesting the next batch.

* changed text for step away reasons  - #163 (#163)

* Change the wording for texts according to suggestions - #162 - (#162)

* feat: dynamic step away view

* feat: dynamic step away view

---------

Co-authored-by: Radovan Tomášik <tomasik@mail.muni.cz>
RadovanTomik added a commit that referenced this issue Mar 3, 2023
@konradlang @RadovanTomik
fix: wording for stepaway texts on UI (#181)
0f9d0f0 
* Throttling the requests by waiting for x seconds in the range [0,3] before requesting the next batch.

* changed text for step away reasons  - #163 (#163)

* Change the wording for texts according to suggestions - #162 - (#162)

* feat: dynamic step away view

* feat: dynamic step away view

---------

Co-authored-by: Radovan Tomášik <tomasik@mail.muni.cz>
jungwire pushed a commit that referenced this issue Mar 27, 2023
@konradlang @RadovanTomik @jungwire
fix: wording for stepaway texts on UI (#181)
b5089ec 
* Throttling the requests by waiting for x seconds in the range [0,3] before requesting the next batch.

* changed text for step away reasons  - #163 (#163)

* Change the wording for texts according to suggestions - #162 - (#162)

* feat: dynamic step away view

* feat: dynamic step away view

---------

Co-authored-by: Radovan Tomášik <tomasik@mail.muni.cz>
konradlang added a commit that referenced this issue Mar 27, 2023
@jungwire @RadovanTomik @konradlang
fix for multirequest problem for locator and finder (#192)
249ea4b 
* should fix the problem with multirequest for locator and finder

* Hotfix: autologin from the directory (#170)

* added attempt to "login" without prompt if fails redirect to login page

* bugfix only overwrite redirect is no error is in QueryString

Co-authored-by: Emilian Jungwirth <emilian.jung+github@gmail.com>

* feature: acceptance test (#169)

* Documentation cleanup (#172)

* updated documentation

* Update development.md

* fix: not reachable collection status message (#173)

* Publishing to GitHub packages (#174)

* feat: GitHub packages

* fix: maven settings

* fix: pom version

* fix: docker registries

* feat: deploy workflow

* feat: deploy workflow on dispatch

* Fix: trivy image scan

* fix: trivy image scan

* fix: deployment url

* fix: jackson media to 2.35 (#175)

* fix: jackson media to 2.35

* fix: commons-configuration to 1.10

* fix: trivy scan

* fix: acceptance test

* Throttling the requests by waiting for x seconds in the range [0,3] before requesting the next batch. (#176)

* hotfix: not reachable typo

* Fix to add userID for selected queries; add queries for owners without queryID to allow acting on behalf of Biobank; (#179)

for details see RT #7426: AW: [ext] AW: Negotiator requests

* Update build.yml

* Update build.yml

* Update build.yml

* fix: wording for stepaway texts on UI (#181)

* Throttling the requests by waiting for x seconds in the range [0,3] before requesting the next batch.

* changed text for step away reasons  - #163 (#163)

* Change the wording for texts according to suggestions - #162 - (#162)

* feat: dynamic step away view

* feat: dynamic step away view

---------

Co-authored-by: Radovan Tomášik <tomasik@mail.muni.cz>

* feat: mail notifications toggle

Co-authored-by: Konrad Lang <110096044+konradlang@users.noreply.github.com>

* Feat: no basic auth (#183)

* feat: removed auth check

* feat: removed auth check
* fix: tests mocks

* feat: favicon and tag title (#184)

* feat: favicon and tag title

* feat: favicon and tag title for admin

* Revert "Feat: no basic auth (#183)" (#185)

This reverts commit 3546897.

* chore: new favicon

* chore: new favicon (#186)

* chore: new favicon

* chore: workflow hardening

* chore: production deployment workflow

* chore: production deployment workflow

* hotfix: surefire config

* fix: dockerhub credentials

* hotfix: maven settings (#189)

* hotfix: maven settings

* chore: workflow cleanup

* Hotfix/performace owner query load (#188)

* getQuery by ID for owner directly from DB

* redirect to researcher query detail page if not an owner

* only use query_paarm_for redirect

* fix: dockerhub credentials

* removed old commented code

* added check for query_param

---------

Co-authored-by: Emilian Jungwirth <emilian.jung+github@gmail.com>

* Feat/swagger (#187)

* feat: swagger json

* fix: git properties

* chore: workflow java version 11

* feat: swagger-ui

* Update pom.xml to 2.7.6

* update unit tests

---------

Co-authored-by: Radovan Tomášik <tomasik@mail.muni.cz>
Co-authored-by: Konrad Lang <110096044+konradlang@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: wording for stepaway texts on UI BBMRI-ERIC/negotiator-v2
1 participant
@konradlang