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

[Instruments] Correct/Update TestPlan and Readme for DoD feature / linst #7403

Closed
christinerogers opened this issue Mar 19, 2021 · 0 comments · Fixed by #8362
Closed

[Instruments] Correct/Update TestPlan and Readme for DoD feature / linst #7403

christinerogers opened this issue Mar 19, 2021 · 0 comments · Fixed by #8362
Assignees
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Documentation PR or issue introducing/requiring modifications to the code documentation (test plans, wikis, docs)

Comments

@christinerogers
Copy link
Contributor

#4929 introduced a visible bug (fix #7397) in every Linst instrument form which wasn't caught during 2 rounds of release testing (23 spring 2020 and mid-release fall 2020)
Would have been caught if we tested a sample Linst instrument at every release and/or if #4929 updated relevant documentation.

Documentation to be updated:

  • Instrument module Readme : Configuration section
  • Instrument module test plan : at least a basic form save should be tested on a Linst instrument, so that we have a chance to catch issues.
@christinerogers christinerogers added Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Documentation PR or issue introducing/requiring modifications to the code documentation (test plans, wikis, docs) labels Mar 19, 2021
@zaliqarosli zaliqarosli linked a pull request Feb 6, 2023 that will close this issue
1 task
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Documentation PR or issue introducing/requiring modifications to the code documentation (test plans, wikis, docs)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants