https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42325 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dpavlin@rot13.org Component|Architecture, internals, |Authentication |and plumbing | --- Comment #5 from David Nind <david@davidnind.com> --- Hi. Welcome, and thanks for the patch! There is a bit of a process to get code changes to Koha into a release. Some resources to help you: - Our development workflow: https://wiki.koha-community.org/wiki/Development_workflow . From initial reporting, issues go through a workflow . someone writes the code and a patch . it gets signed off by someone else . goes through our QA team . and is then reviewed by the release manager to get into the next release - Development environment: we don't use pull requests or merge requests (and the GitHub repository is only a mirror), instead we use a development environment (koha-testing-docker, or KTD for short) https://gitlab.com/koha-community/koha-testing-docker - Commit messages: we have some guidelines for patch commit messages https://wiki.koha-community.org/wiki/Commit_messages - the main thing is to format it correctly and have a test plan (the test plan is the key, and should be aimed at a general audience - we have a range of people that test bugs including library staff, not just developers who are familiar with or can work out the code). This is obviously a lot to set up for a one-off or occasional change to Koha! There is also the Development channel in our chat tool to ask for help https://chat.koha-community.org/ (registration required). In the interests of helping out, here is my suggestion for the commit message, and I'm happy to update the commit message. --- Bug 42325: Update text for error message and code comments This patch updates the text for an error message and code comments in Koha::AuthUtils (provides utility functions for managing user passwords). Test plan: 1. Review the diff and proposed update to the error message, and make sure it is correct. 2. Review the diff and the proposed updates to the code comments, and make sure they are correct. --- As this is related to authentication (note: I'm not a developer!), I've changed the Koha component to that. Also, as this is something that requires a developer to review, then I'd add a comment to the bug saying signoff required by a developer (as you need to understand the code, rather than work through steps and workflow logic using the staff interface or OPAC). Apologies if this is a lot! 8-) David -- You are receiving this mail because: You are watching all bug changes.