Feature #7275
closedAdd help text for DNS Made Easy
0%
Description
src/usr/local/www/services_dyndns_edit.php
domainname field: Add "DNS Made Easy: Dynamic DNS ID (NOT hostname)"
username field: Add "DNS Made Easy: Dynamic DNS ID"
password field: Add "DNS Made Easy: Dynamic DNS Password"
Updated by Phillip Davis almost 8 years ago
@Jeremy, since you know exactly what data should go in the fields, it will be easy if you go to:
https://github.com/pfsense/pfsense/blob/master/src/usr/local/www/services_dyndns_edit.php
make edits to the text and keep clicking through GitHub to make a pull request.
Updated by Jeremy Nelson almost 8 years ago
I didn't know if I should directly make a pull request to master, and quite frankly, I'm not all that familiar with git, so that link made it super easy. Thanks guys! Love the project - I look forward to contributing here and there. These little usability things can make a big difference for a new user.
Updated by Jeremy Nelson almost 8 years ago
I guess I'm not sure what I should do next - do I need to link this to Github pull request 3554 somehow, and/or mark this as ready for review?
Updated by Phillip Davis almost 8 years ago
Mentioning the Pull Request is good enough. I usually paste the whole link to it, like this:
https://github.com/pfsense/pfsense/pull/3554
Then people can click on it directly from here.
Look at the pull request - it is mentioning that you need to "sign" the Contributor Licensing Agreement. Click the link and follow the bouncing ball to do that.
Updated by Jim Thompson over 6 years ago
Phillip Davis wrote:
Mentioning the Pull Request is good enough. I usually paste the whole link to it, like this:
https://github.com/pfsense/pfsense/pull/3554Then people can click on it directly from here.
Look at the pull request - it is mentioning that you need to "sign" the Contributor Licensing Agreement. Click the link and follow the bouncing ball to do that.
you no longer have to sign the CLA, assuming the package and your changes are licensed under Apache 2
Updated by Jim Pingle over 5 years ago
- Project changed from pfSense Packages to pfSense
- Category set to Dynamic DNS
- Status changed from New to Resolved
PR was merged years ago