Closed sbeaver-netgate closed 9 years ago
Looks good! If you want to create a PR I"ll merge it.
OK
On Mar 26, 2015, at 9:33 AM, SjonHortensius notifications@github.com wrote:
Looks good! If you want to create a PR I"ll merge it.
— Reply to this email directly or view it on GitHub https://github.com/SjonHortensius/pfsense/issues/40#issuecomment-86514810.
I guess emoji don’t work when viewing a .md file :(
On Mar 26, 2015, at 9:33 AM, SjonHortensius notifications@github.com wrote:
Looks good! If you want to create a PR I"ll merge it.
— Reply to this email directly or view it on GitHub https://github.com/SjonHortensius/pfsense/issues/40#issuecomment-86514810.
Maybe referencing the image directly works?
Why not just use the emoji as character? Chrome on OS X supports it; not sure if it's also supported on other platforms though. Check http://www.fileformat.info/info/emoji/browsertest.htm .
✅ 🙈
This looks great! But I don't really want to submit a pull request every time I'm starting on a page, can this go in a wiki instead?
Sure. Sjon asked me to commit it, but I will be putting it in a Wiki as well later today (after I have made the emoji work properly :)
I will also be starting a Wiki section to document units tests. Each time we convert a GUI page I’d like there to be some notes on how to test the features on that page. Since we are touching every php file anyway now seems like a good time to start that practice. With any luck by the time we get to the end of the project we will have a complete set of test cases that we can use for regression testing in the future.
Steve
On Mar 27, 2015, at 11:43 AM, Thane Gill notifications@github.com wrote:
This looks great! But I don't really want to submit a pull request every time I'm starting on a page, Can this go in a wiki instead?
— Reply to this email directly or view it on GitHub https://github.com/SjonHortensius/pfsense/issues/40#issuecomment-86980020.
@psophis If you prefer that I can move it to a wiki. I value the progress more for its checkboxes so we have an overview of what is done and what not. As for the claiming of pages for conversion, I think the issues we've used so far work fine
@sbeaver-netgate If you want to document your testing; go ahead, but I think it doesn't belong in this repo which is focused on converting to Bootstrap :)
Hi Sjon,
OK. I’m happy to do whatever you prefer. Just a suggestion. I will be creating unit test documents as I work on each page and was thinking about the best place to house those. A Wiki page on pfSense/bootstrap seems like the best place, but I can also use a wiki page at wiki.pfsense.org http://wiki.pfsense.org/ or anywhere else JimT prefers.
Thanks for tolerating my “work” this week. As I’m sure you can tell you can tell this is my first exposure to git and/or GitHub( I am a long time SVN and AccuRev user) :(
I’ll figure it out before too long.
Stephen Beaver | +1 (512) 646-4100;239 | sbeaver@netgate.com | www.netgate.com
On Mar 28, 2015, at 8:17 AM, SjonHortensius notifications@github.com wrote:
@psophis https://github.com/psophis If you prefer that I can move it to a wiki. I value the progress more for its checkboxes so we have an overview of what is done and what not. As for the claiming of pages for conversion, I think the issues we've used so far work fine
@sbeaver-netgate https://github.com/sbeaver-netgate If you want to document your testing; go ahead, but I think it doesn't belong in this repo which is focused on converting to Bootstrap :)
— Reply to this email directly or view it on GitHub https://github.com/SjonHortensius/pfsense/issues/40#issuecomment-87217758.
I think tests will be very beneficial, but I don't have time for it (I do obviously test my changes, but documenting procedures takes time). I'd rather focus on the code.
I understand getting used to git takes some time, don't worry about it. :+1:
Bootstrap Conversion Progress
Use this table to list the PHP pages that have been converted to bootstrap, those that have yet to be converted, and those that are currently being worked on.
The table can be generated by running this bash script on the directory in which the php files are located. e.g.:
You can then copy/paste the script output into this file. (Is that recursive?)
Use the :white_check_mark: emoji (
:white_check_mark:
) to show that the bootstrap conversion is complete Use the :arrows_counterclockwise: emoji (:arrows_counterclockwise:
) to show that the bootstrap conversion is in progressPHP file status