Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Let's Try This Again... #2

Merged
merged 84 commits into from Jan 13, 2013

Conversation

Projects
None yet
2 participants
Contributor

nathanh89 commented Jan 9, 2013

Now that my commit history is attached, I formally request you pull on my fork. I will not stop making that joke. Also, this is irrelevant but I am typing so much faster than the computer is rendering right now I'm about thirty seconds ahead.

BudgetBoss works here, mostly. It's a good jumping off point (I think) for future changes under your more direct supervision..and with the branch/change/merge method of adding new features it's a good place to start. Anyway I've spent enough time playing around with git in the command line, I'm gonna go read now.

nathanh89 added some commits Nov 8, 2012

Choosing save/load directories works well, as does saving itself. Loa…
…ding not yet implemented. Needs some serious refactoring.
File operations are pretty much done, for now. Can open/load Budgets …
…from any directory where they've been saved, can save modified or new ones in any directory.
Fixed bug that allowed users to save to a directory not ending in / m…
…essing up the filenames and desired save directory
MainMenu has begun! Currently you can only save or exit, though. Remo…
…ved the autoSave method from the end of the main program, and changed that method name to saveBudget
Basic Main Menu and Main Editor Menu functionality is in place. Need …
…to spend some time refactoring at this point
Added MainMenuOption interface, adapted main menu methods to an array…
… of MainMenuOptions, user input is passed as index to call desired option. Going to do the same thing to the editor
Modified YOrN vlidator and Path validator, modified Salvation to work…
… with new methods. Need to fix Opener and TheCreator
Close to finishing refactor. Code currently not working, but it's bee…
…n an hour since my last commit, so...being careful
Code seems to be working, brining a potential end to the massive refa…
…ctoring. At this point though, I think it would be prudent to add some tests to be sure and some more console tests will probably be run as well
Worked the lastfew known bugs out of the code from the refactor. Code…
… is working! FUCK YA! So far as I know. Must write more tests tomorrow.
Updated To-Do list, seperated implemented features, added FeatureRequ…
…ests to record exact wording of requested features
Replaced String dates in Budget with LocalDate from Joda-Time, fixed …
…get/set/print methods that broke, added class DateConverter
Added MasterMenu interface so both (and any future) menus/menu select…
…ions can be validated with the same InputValidator method, validatesMenuChoice
Made Budget extend MasterBudget, added several classes/test cases to …
…work on today for expanding Budget functionality
Adding ability to load different Budget in same instance of BudgetBos…
…s, modified the menu, updated some CLI prompts, added 0 check for menu choices
BudgetBoss MainMenu comes up on program launch now, need to add some …
…checks to make sure you can't edit the defaultBudget, but for now you can load a budget, then load a different one
You can exit from initial MainMenu, or load a budget. Anything else w…
…ill prompt you first to open a saved budget, then to create one
Removed methods no longer necessary in Opener and Creator, separated …
…process of opening a budget from creating a budget on the MainMenu
Removed last y/n prompt, and removed validator method. Salvation now …
…directly prompts for save directory, removed irrelevant test
Sets default direcotry on successful save, shows currently opened bud…
…get on editor menu, added global 'exit' command to end program
Moved crappy check from InputValidator to individual menu options so …
…I don't have to mess with the Validator every time I add a menu option
Invalid menu entries now trigger a reprint of the menu with the error…
… message at the bottom, prompting for another input
About to refactor global exit command and I'm mildly afraid. So I'm c…
…ommitting whatever small tweaks I've done to the code in the last couple hours
Added ability to differentiate between Windows and Linus environments…
… in path generation. I'll be working on Windows 7 now.

leadVisionary added a commit that referenced this pull request Jan 13, 2013

Merge pull request #2 from nathanh89/master
Thanks for the contribution!

@leadVisionary leadVisionary merged commit 46b4f4e into visionary-software-solutions:master Jan 13, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment