Skip to content

Commit

Permalink
Bug SAFeSEA#34/SAFeSEA#35, update CHANGELOG; style Uptime controller …
Browse files Browse the repository at this point in the history
…[iet:10301360]

* [iet:10302190][ci skip]
  • Loading branch information
nfreear committed Feb 12, 2018
1 parent a39c280 commit 9ea17ba
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 9 deletions.
33 changes: 25 additions & 8 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,17 @@

# OpenEssayist changelog

## [Version 3.0.0](https://github.com/IET-OU/openEssayist-slim/releases/tag/3.0.0-beta)
## [Version 3.0.1][v3.0.1-beta]

* Date: ~ 7 February 2018 _(Development: 10 Jan - 6 Feb.)_
* Date: _~ 14 February 2018_,
* Add database column `users.visit_count`; record user's login (Bug #35),
* Fix for user-roles in SAMS authentication,
* Fix user-interface texts in templates (_My assignments_, _Feedback_),
* Fix handling of database connection errors.

## [Version 3.0.0][v3.0.0-beta]

* Date: 7 February 2018 _(Development: 10 Jan - 6 Feb.)_
* Tag: _3.0.0-beta_ (Describe: `v2.6-111-g71a6a5f`)
* Notes: [Google Docs][]
* Built on: PHP 7.0 / Slim framework v 2.2.0 / MySQL
Expand All @@ -13,9 +21,10 @@ Below are the __headline__ items. More details are in [Google Docs][].

### User-interface

* Added configurable `maxlength` character limit to the draft-essay submission form — prevent analyser errors; reduce server-load (Bug #30)
* Added configurable `maxlength` character limit to the draft-essay
submission form — prevent analyser errors; reduce server-load (Bug #30)
* Made error messages to user more user-friendly — for example, prefixed them with "_Sorry. ..._",
* _NO_ other new user-interface features/functions — apart from _SAMS authentication_ below.
* _NO other new user-interface features/functions — apart from SAMS authentication below._

### PHP

Expand All @@ -25,7 +34,8 @@ Below are the __headline__ items. More details are in [Google Docs][].
* Configuration — make `rd_save_path`, `analyser_url` etc. configurable (Bug #23)
* Added [Travis-CI][] continuous integration, with PHP linting [QA] (maintainability) (Bug #20)
* Moved core _EssayAnalyser_ wrapper code to separate class (maintainability) (Bug #32)
* Moved a lot of `require` and `Twig` configuration from front controller (`index.php`) to separate PHP files (maintainability) (Bug #31)
* Moved a lot of `require` and `Twig` configuration from front controller
(`index.php`) to separate PHP files (maintainability) (Bug #31)
* Moved database connection code from `app/config.php` to separate class (Bug #31)
* Broke model classes into separate files (find-ability),
* Added missing logging; improved error-handling (debugging aid, analytics),
Expand Down Expand Up @@ -57,23 +67,24 @@ Below are the __headline__ items. More details are in [Google Docs][].
* Added _Nicolas Van Labeke_ (`vanch3d`) to `composer.json`
* Removed legacy `composer.phar` binary from Git; removed `./openessayist.sql.zip` (2.6 MB)
* Moved unused Javascript from Twig templates to separate JS file(s),
* Added test data — sourced from [Project Gutenberg][pg] and [ORO][].
* Added test data — sourced from [Project Gutenberg][pg] and [ORO][],
* Documentation, changelog (Bug #34).


---
## [Version 2.7](https://github.com/IET-OU/openEssayist-slim/releases/tag/v2.7)

* Date: 24 February 2014.
* Tag: _v2.7_ ("_v2.7 beta_")
* Commit: [IET-OU/openEssayist-slim `7a3d3861`](https://github.com/IET-OU/openEssayist-slim/commit/7a3d3861e7cdb834962e82902c4d2f4e3f0d50b9)
* Commit: [IET-OU/openEssayist-slim `7a3d3861`][v2.7-co]

> version 2.7 - ready for Phase II Evaluation
## [Version 2.6](https://github.com/IET-OU/openEssayist-slim/releases/tag/v2.6)

* Date: 27 September 2013.
* Tag: _v2.6_
* Commit: [IET-OU/openEssayist-slim `e6f00deb`](https://github.com/IET-OU/openEssayist-slim/commit/e6f00debfaf948d966f443cf8b0a24947e6c6f81)
* Commit: [IET-OU/openEssayist-slim `e6f00deb`][v2.6-co]

> version 2.6 - ready for September evaluation
Expand All @@ -91,6 +102,12 @@ Below are the __headline__ items. More details are in [Google Docs][].
[ou]: http://www.open.ac.uk/
[iet]: https://iet.open.ac.uk/

[v3.0.1-beta]: https://github.com/IET-OU/openEssayist-slim/releases/tag/3.0.1-beta
[v3.0.0-beta]: https://github.com/IET-OU/openEssayist-slim/releases/tag/3.0.0-beta
[v3.0-co]: https://github.com/IET-OU/openEssayist-slim/commit/e8aa09d4a9f809741fd927b68a92e231e0317f52
[v2.7-co]: https://github.com/IET-OU/openEssayist-slim/commit/7a3d3861e7cdb834962e82902c4d2f4e3f0d50b9
[v2.6-co]: https://github.com/IET-OU/openEssayist-slim/commit/e6f00debfaf948d966f443cf8b0a24947e6c6f81

[releases]: https://github.com/IET-OU/openEssayist-slim/releases
[readme]: https://github.com/IET-OU/openEssayist-slim#readme
[pkg]: https://github.com/IET-OU/openEssayist-slim/blob/3.x/package.json#L12-L38 "package.json"
Expand Down
3 changes: 2 additions & 1 deletion app/controllers/uptime.controller.php
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,8 @@ protected function renderStatusPage( $success, $http_status, $message, $raw ) {
<html class="<?= $success ? 'ok' : 'error' ?>" data-stat="<?= $http_status ?>" data-raw="<?= $raw ?>" lang="en">
<meta name="robots" content="noindex">
<title> OpenEssayist — status (uptime) </title>
<style> body { background: #fefefe; font: 1em sans-serif; margin: 3em auto; max-width: 45em; min-width: 20em; text-align: center; } .error .m { color: #d00; } .f { margin: 4em; padding: 1em; border-top: 1px solid #bbb; } </style>
<style> body { background: #fefefe; font: 1em sans-serif; margin: 3em auto; max-width: 45em; min-width: 20em; text-align: center; }
.m{ padding:1em; border:1px solid; } .error .m{ background:#fee; color:#d00; } .ok .m{ background:#dfd; border-color:#bdb; color:#050; } .f{ margin:4em; padding:1em; border-top:1px solid #bbb; } </style>
<link href="<?= $baseUrl ?>assets/openessayist/img/favicon.ico" rel="shortcut icon">

<p class="m"> <?= $message ?> <small>(<?= $http_status ?>)</small> </p>
Expand Down

0 comments on commit 9ea17ba

Please sign in to comment.