It would be nice to set the estimated read time manually. Since the module already uses a field, we can use a switch in the field widget in the same style of, e.g., pathauto alias or menu item widget.
Depends on #3377695: Introduce services to make testing easier!
Review merge request.
Add tests after #3377695: Introduce services to make testing easier.
The estimated read time field widget offers a switch to manually set the read time.
Changes the estimated read time field schema and adds a boolean field auto. After updating the module a database update is required.
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #3
simonbaesePlease do not forget to
updbbefore testing.Comment #4
simonbaeseRerolled after #3361590: Estimated Read Time module make webform send emails twice.
Comment #5
simonbaeseComment #6
simonbaeseActually putting this issue back to pp-1, because it will make sense to add test coverage after #3351967: Add kernel tests!
Comment #7
simonbaeseComment #8
simonbaeseWill update the MR with the recent changes soon.
Comment #9
simonbaeseComment #10
mtalt commentedComment #12
mtalt commentedLooks great @simonbaese. Thank you!