0.9.1 release
Issue to tag release stuff to
Comments (30)
-
reporter -
reporter Merged in issue749minordocsupdates (pull request #302)
Docs updates preparing for 0.9.1
→ <<cset 5e8641770fcb>>
-
reporter Adding INFO/DEBUG switch back to prevent strange logging errors in testing. Refs
#751→ <<cset e704f05f2cc6>>
-
reporter Adding headline changes to release notes. Refs
#751[skip ci] docs only→ <<cset e6b0339ceb72>>
-
reporter - edited description
-
reporter Added catch-all to changes. Refs
#751[skip ci] docs only→ <<cset f57b229a6fd0>>
-
reporter Attempt to update offline upgrade for 0.9.1. Refs
#751[skip ci] docs only→ <<cset cd6e2d815026>>
-
reporter Teaks to offline upgrade doc. Refs
#751[skip ci] docs only→ <<cset 87648787e18c>>
-
reporter Improvements to layout. Refs
#751[skip ci] docs only→ <<cset 90b4352135d3>>
-
reporter Updated offline-install for 0.9.1. Refs
#751[skip ci] docs only→ <<cset 25d26822db26>>
-
reporter Fixing all instances of celery 4.3 to 4.2.2. Refs
#751[skip ci] docs only→ <<cset cde065401f56>>
-
reporter Forgot zip for Physics orthanc. Refs
#751[skip ci] docs only→ <<cset 69038451dbc8>>
-
reporter Updating orthan config docs with current orthanc lua file content, plus adding suggestion of installing zip. Refs
#751[skip ci] docs only→ <<cset ab5de0238324>>
-
reporter Bumping all the version numbers for 0.9.1b1. Refs
#751→ <<cset 3ce41eee316e>>
-
reporter @David Platten @Luuk I have pushed out a first beta for 0.9.1. I am aware that the following need to be resolved:
- Upgrade procedure for old Python versions on Windows, particularly when using virtualenv. What I have written worked for me, but ongoing discussions with David suggest that it will be more complicated.
- Ideally, scripts for daemonising Celery on Windows will be modified as Celery 3.1.25 is more stable than other versions have been.
If you have any other comments or questions (anyone!) please shout.
Docs for beta release are at https://docs.openrem.org/en/0.9.1b1-docs
-
reporter - edited description
Issue to tag release stuff to
-
reporter - edited description
Issue to tag release stuff to
[https://docs.openrem.org/en/0.9.1b1-docs/](https://docs.openrem.org/en/0.9.1b1-docs/)
-
reporter - edited description
Issue to tag release stuff to
[https://docs.openrem.org/en/0.9.1b2-docs/](https://docs.openrem.org/en/0.9.1b2-docs/)
-
@edmcdonagh I tested version 0.9.1 beta 2 using my test environment (virtual python environment using virtual directory for the website)
-
Upgrade went smoothly (I already had a correct python and celery version)
-
We might want to add to the instructions that if you use a virtual directory you should run
-
python manage.py collectstatic_js_reverse (does the default installation has an updated version of reverse.js?)
- btw the new version of django-js-reverse (0.9.1., is this a coincidence or not?) does contain the improvement taking into account the ROOT_URLCONF.
- So you might also want perform pip install collectstatic_js_reverse -U before
- And then the documentation for virtual directory should be updated as you don’t need to change the code yourself any more
-
-
-
No issues concerning import and export of data
-
Web page issues:
-
The “Modify fluoroscopy high dose alert settings” page has hard-coded links (buttons), so not working in virtual directory
- highdosemetricalertsettings_form.html, line 46
- See issue models.py
-
The “Flouoro allert notifications settings” page has a hard-coded link (button), so not working in virtual directory
- rfalertnotificationsview.html, line 33
-
The “modify skin dose map settings” page has a hardcoded link (button), so not working in virtual directory
- see issue models.py
-
models.py contains hardcoded urls
- line 98, 114, 144
-
-
Minor issue:
- The pagination in the review page looks different than on the modality filtered pages
I think that’s it. So furthermore it looks great!
-
-
reporter Brilliant, thanks Luuk.
Are you able to do a PR for the reverse js stuff?
-
You mean the documantion changes for the reverse js stuff? Or also an update version of the default reverse.js?
And, yes, I can do that.
-
reporter Fixed the first of the URLs pointed out by Luuk. Tested. Refs
#751[skip ci] web interface not tested :-(→ <<cset 96f2a0fe744d>>
-
reporter Modified all the other hard coded urls. Tested without virtual directory. Refs
#751[skip ci] web interface not tested :-( Workload absolute URL in models.py (line ~144) does not get used, and was wrong. Confirmation would be appreciated!→ <<cset 2c807d360ac2>>
-
reporter @557058:23c225ec-4b40-4343-b122-aabbe8eaac40 - are you able to confirm the statement about the
HomePageAdminSettings
get_absolute_url
being unused? I have commented it out in theissue751LuukTesting
branch, and I think everything is still working.That was meant to be @dplatten
-
@edmcdonagh I tested your changes using a website with virtual directory and they work fine! Also the Django pagination on the review page looks fine!
-
reporter Thanks @LuukO will you approve pull request #306?
-
reporter -
reporter Version number bump to 0.9.1b3. Refs
#751→ <<cset 79d4331160dc>>
-
reporter Bump version for 0.9.1 release. Refs
#751→ <<cset 471ae7cbe8f7>>
-
reporter - changed status to resolved
0.9.1 released 16th May 2019
- Log in to comment
Realised the issue, refs
#751[skip ci] docs only.→ <<cset 7dfe5d7e7c16>>