Refactor teal.reporter (part 2)

[donyunardi]

Continuation of #104

From comments: #104 (comment)

---

Done

teal.modules.general

• Rewrote example modules to align with the new design
  👉 POC Redesign of teal.reporter teal.modules.general#862
• Ensured reproducibility of reports
  👉 Improve the Reproducibility Mechanisms and Code Display in Report Output teal.reporter#280
• Extended teal.code::eval_code and teal.code::get_code to support code labelling
  👉 introduce label to eval_code, within and get_code teal.code#248

teal

• Exposed reporter as an init parameter to allow global disabling
  👉 [Feature Request]: Globally disable reporter feature teal#871
• Added reporter$set_template() for setting custom templates when users add cards
  👉 [Feature Request]: User-defined templates for reports teal.reporter#271
• Introduced modify_teal_module_output() so devs can modify or disable report cards
  👉 [Feature Request]: Allow app dev to supply card function. teal.modules.general#736
  👉 Also allows disabling reporter on a per-module basis
  👉 optional reporter in the independent teal modules teal#678
• Added add_document_button_server() with a standalone "+" button (no more old buttons in modules)
  👉 https://github.com/insightsengineering/coredev-tasks/issues/545

teal.reporter

• Added report_document() and edit_report_document() for the new ReportDocument S3 class
• Extended Previewer and Reporter to support ReportDocument
• Deleted private Rendered class
• Refactored:
  • report_render_and_compress
  • report_render
  • Added load report button to previewer buttons

---

In Progress

teal

• Finalize placement of the "+" button
  👉 POC Redesign of teal.reporter teal#1499 (comment)

teal.reporter

• Editor
  • Work in progress 👉 Edit Functionality for the Report Previewer module teal.reporter#312
  • User request 👉 Showcase functionality of editor for report previewer teal.reporter#306

---

🔜 TODOs

teal

• [vignette] Create vignette for modify_teal_module_output() teal#1523 - Documentation (13 hours)
  • Add documentation and a vignette for modify_teal_module_output()
• Create wrapper for modify_teal_module_output and export them teal#1521 - Wrappers (13 hours)
  • Consider which wrappers to expose for modify_teal_module_output() (e.g. disable_teal_module_report_card)
• Viewing the structure of the report document for a module teal#1522 - Print
  • Consider extending print of modules to include report_card body
    👉 POC Redesign of teal.reporter teal.modules.general#862 (comment)

teal.reporter

• Fix known issues:
  • [Feature Request]: Dealing with duplicate card name teal.reporter#313
    Do not allow to create cards with the name that already is used (we aim not to allow user to input the same name, card name is the same with card title, UI needs to be updated to handle this assertion as well in the modal when we add the card) 5 hours
• [Documentation] Vignette for new feature teal.reporter#314
  Write a vignette covering new features (explain the new card object (ReportDocument), tutorial, and how they can be extended in teal, etc) - 8 hours
• Soft Deprecate old methods teal.reporter#315
  Soft Deprecate old methods (possibly just 1 parent method, ContentBlock method. we may need to create an initialization method on the parent class and put the deprecation warning there, make sure to test) - 3 hours
• Ensure R CMD Check is green teal.reporter#316
  Fix failing tests (R CMD check fails: unit test, examples, package metadata, a bit of everywhere) - 13 hours
• Further refactoring:
  • Reuse opportunities between download_report_button_srv and reporter_previewer_srv teal.reporter#317
    Check for code reuse opportunities between download_report_button_srv and reporter_previewer_srv (8 hours)
    👉 Redesign of teal.reporter Cards teal.reporter#307 (comment)
• Editor improvements
  • [Feature Request]: Rearrange the edit/delete button teal.reporter#318
    moving card edit/delete buttons somewhere else (UI to find a better place for the buttons) - 5 hours
  • [Feature Request]: Allow to edit card title teal.reporter#319
    allow to edit card title (make sure that the card name still unique, ) - 8 hours
    👉 Edit Functionality for the Report Previewer module teal.reporter#312 (comment)
  • allow to sort blocks (drag and drop the blocks) - Discussed with @kumamiao , this is not needed.
  • allow to delete blocks - solution is being rewrite with a better approach, will need to circle back on this.

Module Packages

• Rewrite report cards based on the new design in:
  • Documentation on module's return teal.modules.general#875 (3 hours, only when the teal.reporter is ready)
  • Rewrite reporter code in modules teal.modules.general#874 - 15 modules (23 hours)
  • Rewrite reporter code in modules teal.modules.clinical#1376 - 36 modules (54 hours)
  • Rewrite reporter code in modules teal.osprey#299 - 10 modules (15 hours)
  • Rewrite reporter code in modules teal.goshawk#342 - 6 modules (9 hours)
  • Rewrite reporter code in modules teal.modules.hermes#413 - 8 modules (12 hours)

---

Future Plans

• Expose Show R Code as a button next to Add to Report button
  • This will require the module to return teal_data/qenv
  • This will require srv_teal_module to depend on whether module returned teal_data/qenv to show the button
  • This will allow to modify the teal_data/qenv returned by the module, so that Show R Code can be also customized or nullified