search

PMR2 / pmr2.app

Physiome Model Repository 2 (PMR2) core package.
https://www.physiomeproject.org/
1 stars 1 forks source link

Provide link to report issues with exposures #9

Closed metatoaster closed 2 years ago

metatoaster commented 2 years ago

Given that the issue tracker at this GitHub repository is being used to track issues to do with models in PMR, it would be good to provide a link from the exposure to a prefilled template to file the relevant issue they may have found.

metatoaster commented 2 years ago

This feature has been created and deployed onto the staging instance.

Please feel free to have a play with it as is, people who have interacted significantly or have filed a large number of issues may be interested in providing a comment or opinion about this feature, such as the wording of the link and associated form on the issue tracker.

cc: @nickerso @bilalshaikh42 @MichaelClerx @jonrkarr

MichaelClerx commented 2 years ago

Thanks @metatoaster ! It looks good, but you might want to change the automatically set title for exposure issues? The URI already contains the exact exposure, so this could perhaps be something less scary like Problem with Beeler, Reuter, 1977 (workspace name), or Problem with "Reconstruction of the action potential of ventricular myocardial fibres" (exposure name).

Similarly, I suspect the Report an issue with this exposure button will go untouched by most users, who are blissfully unaware of what an exposure is. (Semantically it doesn't make sense either, you wouldn't report an issue with a git tag, but an issue with the stuff stored at the commit that the tag references). This could just be "Report a problem with this model", I think. (Sure, an exposure can technically expose something other than a model, but unless we're also renaming it the Physiome Exposure Repository I feel "model" should be general enough in this case)

metatoaster commented 2 years ago

Modified the wording to "resource" to be even more generic to work around the "other than a model" issue, and it is now staged, along with the title change.

metatoaster commented 2 years ago

This was released yesterday onto production.