ostroffjh (Jack Ostroff)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Nov 6 2016, 5:21 PM (124 w, 1 d)
Availability
Available

Recent Activity

Feb 14 2019

ostroffjh added a comment to D18789: Add support for assigning tags in transaction splits editor.

If tags are always assigned to splits, then does a tag being shown for a transaction simply mean it is attached to all splits of the transaction? If so, then it might be nice to show for the transaction all tags attached to any splits, with some indication whether the tag is attached to all splits or only some (bold or not, different colors, different font, ...?)

Feb 14 2019, 5:43 PM

Feb 7 2019

ostroffjh added a comment to D18789: Add support for assigning tags in transaction splits editor.

Is it intended that there can be tags on the transaction itself, and also on each split? (This is OK - I just want to be sure it is what is intended.)

Feb 7 2019, 6:55 PM

Jan 29 2019

ostroffjh added a comment to D18590: Fixes "In German, the dialog uses "share" if it is actually shares and/or bonds (i.e. securities)..

I certainly can't comment on the German, but as long as this is only changing the German translation, I have no problem with it.

Jan 29 2019, 4:33 PM · KMyMoney

Jan 27 2019

ostroffjh added a comment to D18568: Don't die when finding an unknown email.

So, if the person made the original commit by Phabricator, he can request his addition to the script the same way.
Separately, there sill is the case (don't know if real or only potential) of someone making a contribution by sending to a mailing list, so he might not actually have any KDE account at all. I suppose that in that case, if the person had any interest in KDE, it would be reasonably to request he get a basic account (identities.kde.org, I assume).
Finally, would there be any reasonable way to use that basic KDE accountname as a username? (Am I correct a dev account is just a basic KDE account which is in teh developers group? If so, we know there wouldn't be any overlap in names.) I don't think we want a file with every kde account listed, due to the size - might there be an "authors" group for such purpose? I'm just afraid that it wouldn't be easy to maintain. It would need anyone listed as a git author, but who is not already in the developer group.) For now, I'd say let's get the current proposal committed, then we can discuss the additional issues.

Jan 27 2019, 10:40 PM
ostroffjh added a comment to D18568: Don't die when finding an unknown email.

Being Perl, It could be done either way. The current code just repeats creating the hash item, which is essentially a no-op. If you use a list, you would have to check if the email is already on the list before pushing it onto the list. (You could always sort and remove dups before printing, but the hash seems simpler to me.

Jan 27 2019, 8:24 PM
ostroffjh added a comment to D18568: Don't die when finding an unknown email.

Looks good to me. I had also been considering the possibility of an authors-accounts file (along with accounts and disabled-accounts) but I don't know that it would add anything to what's already proposed here, and maintaining that file would probably need to be done manually.
One problem remains, however. If such a listed author agrees to be added to the script - what should be used as username, since there isn't an account? Would simply adding the email address instead of username be reasonable?

Jan 27 2019, 8:17 PM

Jan 25 2019

ostroffjh added a comment to D15215: Support sell w/o asset account.

Other than allowing the transaction to be saved without specifying a brokerage account, what would need to be tested? Maybe I'm not sufficiently paranoid, but what effects on reporting are you considering possible>?

Jan 25 2019, 11:17 PM

Jan 22 2019

ostroffjh committed R230:0d62350388d7: Add myself (ostroffjh) to relicenscheck.pl (authored by ostroffjh).
Add myself (ostroffjh) to relicenscheck.pl
Jan 22 2019, 11:44 PM

Dec 30 2018

ostroffjh added a reviewer for D17871: Fix division by zero errors in QueryTable: KMyMoney.

I think this should add the team as reviewer.

Dec 30 2018, 4:18 PM

Nov 11 2018

ostroffjh added a comment to D14522: Integrate Alkimia.

There are several people who are against this, and only one in favor. Would you please consider others' opinions, and defer this further, or put it in a separate branch?

Nov 11 2018, 5:17 PM · KMyMoney

Oct 27 2018

ostroffjh added a comment to D15483: Update the name of the OFX plugin to its new name.

Successfully tested with xmlstorage. I hope to test sql storage within a few days, but I don't actively use sqlstorage. Using actual SQL code would be good, but probably not enough difference in efficiency to worry about in the short term.

Oct 27 2018, 8:31 PM

Sep 2 2018

ostroffjh added a comment to D15215: Support sell w/o asset account.

I have not tested this yet, but I do have one investment account that does this, and I just enter it as a normal sell transaction, with fees of the exact amount of the sale, so the final value of the transaction is zero. I suppose this would make entering that type of transaction easier/fewer steps. (I also think it might be used for equities other than mutual funds.) What I can't tell just looking at this Diff is whether this is to be a new type of transaction, or else how the user indicates that this is the case.

Sep 2 2018, 2:13 PM

Aug 30 2018

ostroffjh added a comment to D8267: Correcting symlinks between col- and row-resize and split_v and _h cursors.

My problem was that I had originally cloned from anongit, so of course any push would fail. I did a fresh clone, "arc patch," and then "arc land" worked just fine.

Aug 30 2018, 11:04 PM · Plasma
ostroffjh committed R113:5bf9dfe906b5: Correcting symlinks between col- and row-resize and split_v and _h cursors (authored by ostroffjh).
Correcting symlinks between col- and row-resize and split_v and _h cursors
Aug 30 2018, 11:02 PM
ostroffjh closed D8267: Correcting symlinks between col- and row-resize and split_v and _h cursors.
Aug 30 2018, 11:02 PM · Plasma
ostroffjh added a comment to D8267: Correcting symlinks between col- and row-resize and split_v and _h cursors.

I tried "arc land" but get
Landing current branch 'fixlinks'.
TARGET Landing onto "master", the default target under git.
REMOTE Using remote "origin", the default remote under git.
FETCH Fetching origin/master...
These commits will be landed:

Aug 30 2018, 10:27 PM · Plasma

Aug 1 2018

ostroffjh added a comment to D14522: Integrate Alkimia.

I'm at least somewhat against this change. Maybe it's just wishful thinking, but if we improve Alkimia, maybe there will be other users. That becomes essentially impossible if the code is moved internal to KMM.

Aug 1 2018, 4:30 PM · KMyMoney

Jul 29 2018

ostroffjh added a comment to D13994: Updated screen shots in user documentation for KMyMoney 5.0.1.

I tried landing this using arc, but got "remote: Audit failure - Commit ab7fee01413bd1870dbcc562b6791721ec4d6064 - Non-full name: Joshua".

Jul 29 2018, 10:52 PM

Jul 28 2018

ostroffjh added a comment to D13994: Updated screen shots in user documentation for KMyMoney 5.0.1.

I'll be glad to land this. Locally it does look like the commit has the correct author, so unless someone says otherwise, I'll land it later this evening.

Jul 28 2018, 8:59 PM

Jul 19 2018

ostroffjh accepted D13994: Updated screen shots in user documentation for KMyMoney 5.0.1.

Thanks for the changes. I think this is good to go. We might come up with further changes, but I'd prefer to at least get the entire set updated to a new and consistent look. You can wait a day or to for any other comments, but I say Ship It.

Jul 19 2018, 8:35 PM

Jul 17 2018

ostroffjh added a comment to D13994: Updated screen shots in user documentation for KMyMoney 5.0.1.

My bad. I misread (or read too fast) your comment (or how much of my message you quoted.) I do need to recompile and try again. Should be OK.

Jul 17 2018, 8:01 PM
ostroffjh added a comment to D13994: Updated screen shots in user documentation for KMyMoney 5.0.1.

@tbaumgart Can you also check the account edit dialog. I just checked, and it is titled new account dialog.

Jul 17 2018, 2:20 PM

Jul 16 2018

ostroffjh added reviewers for D13994: Updated screen shots in user documentation for KMyMoney 5.0.1: ostroffjh, KMyMoney.

This review is for updated screenshots for the manual. I just added myself and the team as reviewers. The image quality looks good overall. Just a few specific comments:

  • account_edit.png looks like it is from the new account wizard, not the edit account dialog. Was it just saved under the wrong name?
  • the kmy file used (in title bar of all full window shots) is screenshots.kmy. While this is not explicitly wrong, it might make more sense to stick with something like MySampleFinances.kmy. (I could send you the one previusly used, or you could create a new one with similar finances but more current dates.)
  • I don't know whether translations also use localized screenshots, but if not, it might be better to use a date format with month name (long or short) to avoid any possible confusion between mm/dd and dd/mm. (this may not matter, I just raise the question.)
  • new.png - I'm not sure where it is used, but the new version looks like an icon for a missing icon, not for adding anything. please confirm.
Jul 16 2018, 9:38 PM

Jun 30 2018

ostroffjh added a comment to D13707: Add field for institution's website URL.

Just a random thought - will any of this code be useful for adding a URL to a Payee? It could be useful there as well.

Jun 30 2018, 5:56 PM

Jun 14 2018

ostroffjh added a comment to D8267: Correcting symlinks between col- and row-resize and split_v and _h cursors.

Fixed those two symlinks and updated the diff. Anything else blocking this?

Jun 14 2018, 7:51 PM · Plasma
ostroffjh updated the diff for D8267: Correcting symlinks between col- and row-resize and split_v and _h cursors.
  • Merge branch 'master' into fixlinks
  • fix cursors/src/symlinks/col-resize and .../row-resize
Jun 14 2018, 7:46 PM · Plasma

Jun 11 2018

ostroffjh accepted D13400: Add csv export doc section.

Overall - I would prefer to see this new section added to the existing csv import docbook file rather than as a new file, but to be sure it does not get forgotten, I think it is OK to add it like this, and I can rearrange when I do the full review if import/export for 5.0.

Jun 11 2018, 10:17 PM · Documentation
ostroffjh added a comment to D13465: Fix xml file saving on Windows.

Sorry - I see this is only supposed to affect Windows, so questions about Linux are unnecessary.

Jun 11 2018, 9:59 PM · KMyMoney
ostroffjh added a comment to D13465: Fix xml file saving on Windows.

Question regarding backporting: Is it correct that this does not affect 4.8 branch at all, or just not the Windows build? I don't know if any Linux distros will need another 4.8 release before being able to fully migrate to 5.x Also, does this need backporting to 5.0 branch or will next release be 5,1 and not 5.0.2?

Jun 11 2018, 9:57 PM · KMyMoney
ostroffjh added a comment to D13293: Provide timezone offset for OFX import.

Suggestion for improved wording for the first sentence in the tooltip in konlinebankingstatusdecl.ui "If the server is located in a different timezone than your local time, then the post date of imported transactions could be off by a day." (I tried to make as an inline comment, but it does not appear to have worked.)

Jun 11 2018, 9:31 PM
ostroffjh added a comment to D13293: Provide timezone offset for OFX import.

I assume the summary should have said different timezone, not different timestamp? I do not have that situation, so I'm not sure I can test, other than to compile and to confirm it does not cause any problems.

Jun 11 2018, 9:28 PM

May 27 2018

ostroffjh added a comment to D12979: Format mymoney headers according to the KDE guidelines.

I'm a little confused - was this committed or abandoned?

May 27 2018, 7:12 PM · KMyMoney

May 19 2018

ostroffjh added a comment to D12979: Format mymoney headers according to the KDE guidelines.

As this is not a really good forum for this type of discussion, I hope I can be sufficiently brief here, and perhaps we can continue the discussion on the dev mailing list.

May 19 2018, 11:05 PM · KMyMoney
ostroffjh added a comment to D12979: Format mymoney headers according to the KDE guidelines.
NOTE: I think it is important to have this discussion, but I do not intend for it to block or delay committing these changes. Even if there is eventual agreement for a different handling of author names, it can be handled with a separate commit at that time.
May 19 2018, 7:18 PM · KMyMoney

May 15 2018

ostroffjh added a comment to D12836: Format headers according to the KDE guidelines.

Thomas: I consider losing track of someone (simply disappearing from the scene) different from someone dying. However, I agree that is a different issue from what to do with listing email addresses which are not know active, or even known to no longer be valid.

May 15 2018, 4:50 PM · KMyMoney

May 12 2018

ostroffjh added a comment to D12836: Format headers according to the KDE guidelines.

Separate question on header style: Is there a suggestion for dealing with a deceased author? (Allan Anderson) There is not much point in listing an email address in this case.

May 12 2018, 5:00 PM · KMyMoney
ostroffjh added a comment to D12836: Format headers according to the KDE guidelines.

From the original summary:

Moreover, there are some names in the headers, which have not event touched the file, so it's false authorship. Those names has been removed. List of the names to remove has been taken from CVS (http://kmymoney2.cvs.sourceforge.net/) and git logs.

I am extremely concerned by this. If two people worked together on writing/editing some code, and only one of them checked the final copy into cvs/git, then the other name would not show up in the logs, but only in the header. Unless you have better proof they did not contribute, I would not remove names.

May 12 2018, 4:56 PM · KMyMoney

Feb 3 2018

ostroffjh committed R261:de9dabedc4e9: finished Accounts chapter in manual for 5.0 (authored by ostroffjh).
finished Accounts chapter in manual for 5.0
Feb 3 2018, 10:07 PM
ostroffjh committed R261:236c5db6c2c8: more doc changes for accounts for 5.0 (authored by ostroffjh).
more doc changes for accounts for 5.0
Feb 3 2018, 10:07 PM
ostroffjh committed R261:059bd6a6a159: Initial update to institutions and accounts docs for 5.0 (authored by ostroffjh).
Initial update to institutions and accounts docs for 5.0
Feb 3 2018, 10:07 PM

Feb 1 2018

ostroffjh committed R261:6a7f9ed6721e: two updated screenshots for 5.0 (authored by ostroffjh).
two updated screenshots for 5.0
Feb 1 2018, 10:51 PM
ostroffjh committed R261:fadb5bf7b150: Update next section of doc (First time use) for 5.0 (authored by ostroffjh).
Update next section of doc (First time use) for 5.0
Feb 1 2018, 10:51 PM
ostroffjh committed R261:86aa0de77f0d: update credits for 5.0 (authored by ostroffjh).
update credits for 5.0
Feb 1 2018, 10:51 PM

Jan 29 2018

ostroffjh committed R261:e5611932b484: add missing XML end tags (authored by ostroffjh).
add missing XML end tags
Jan 29 2018, 4:58 PM
ostroffjh committed R261:f914750dc61f: initial update to whatsnew.docbook for 5.0 (authored by ostroffjh).
initial update to whatsnew.docbook for 5.0
Jan 29 2018, 4:58 PM

Jan 27 2018

ostroffjh added a comment to R261:fbf31e343bba: Update doc license and add first set of new pngs for manual for 5.0.

I've tried running the relicensecheck.pl script. I manually downloaded a copy of kde-common/accounts, but assume I should actually make a local clone - is it in git, or only svn?
The script seemed to stop at the first author email not in the accounts list for index.docbook. Since it said the email wasn't in the accounts list, I assume it got the email from the accounts list - but why did it stop at the first?
The script gave me a long list of needed permissions for whatsnew.docbook (just for an example) (including yurchor, who I'm a bit surprised not to see already on the list).
Is there a standard email I can send to the contributors (or post to the KMyMoney-devel list) which also tells them how to actually grant permission. It looks like I need to do this myself - do I just add myself to the script and then commit and push?

Jan 27 2018, 12:57 AM
ostroffjh committed R261:8556afea536a: Revert FDL->CCYSA4 license change pending permission from contributors (authored by ostroffjh).
Revert FDL->CCYSA4 license change pending permission from contributors
Jan 27 2018, 12:11 AM

Jan 26 2018

ostroffjh added a comment to R261:fbf31e343bba: Update doc license and add first set of new pngs for manual for 5.0.

No, I did not. I will post about this to the kmymoney-devel list. Do I need explicit approval from everyone listed as an author in any of the docbook files, or just provide them an opportunity to object? I'm not even certain that all contributors are still reading the list.

Jan 26 2018, 11:26 PM
ostroffjh committed R261:8d3de09e3609: FDL -> CCBYSA4 for translations (authored by ostroffjh).
FDL -> CCBYSA4 for translations
Jan 26 2018, 10:01 PM
ostroffjh committed R261:fbf31e343bba: Update doc license and add first set of new pngs for manual for 5.0 (authored by ostroffjh).
Update doc license and add first set of new pngs for manual for 5.0
Jan 26 2018, 10:01 PM
ostroffjh committed R261:195351bc6ccb: First commit for updating docs for 5.0. (authored by ostroffjh).
First commit for updating docs for 5.0.
Jan 26 2018, 10:01 PM

Jan 23 2018

ostroffjh added a comment to D10043: Pass MP include header location detected at client configure time to clients.

Sorry - I was moving too quickly, and was thinking this was a change in KMM to handle libalkimia, not a change to libalkimia itself.

Jan 23 2018, 8:07 PM · KMyMoney
ostroffjh added a comment to D10043: Pass MP include header location detected at client configure time to clients.

Thomas - was that a typo for the 5.0 branch?

Jan 23 2018, 6:02 PM · KMyMoney

Jan 22 2018

ostroffjh added a comment to D10001: display reconciliation date on homepage.

This looks better to me. Hopefully Thomas can also check it after solving his crash issue.

Jan 22 2018, 9:39 PM · KMyMoney

Jan 21 2018

ostroffjh added a comment to D10001: display reconciliation date on homepage.

I like this. I have been thinking about filing a wish list for it for some time.

Jan 21 2018, 9:31 PM · KMyMoney

Jan 8 2018

ostroffjh accepted D9591: Prevent creation of investment accounts with opening balance.

OK - if the account type for New Account is set to Investment, the Opening balance is disabled. However, on the next screen, for the brokerage account, there is still not an Opening Balance. Less important, the Account number and IBAN are both disabled. Also, on the final page of the Wizard, there is "Opening balance: 0.00 $" shown for the investment account, and no opening balance for the brokerage account.

Jan 8 2018, 10:15 PM

Jan 7 2018

ostroffjh added a comment to D9710: Allow to load Plugins without installing them.

Normally, I think the idea to be able to test the program from within the build directory is good. However, this has (I think) always been a problem for KDE, which uses environment variables to find many files necessary to run a program. This is why it takes such effort to be able to have both a KDE4 and KF5 version of an application installed at the same time, to be able to use one and test the other. However, thinking about how I do that, I wonder if the same effect couldn't be had here by using a launch script which sets the necessary variables (pointing to the various areas under the build directory) and then launching the app.

Jan 7 2018, 7:52 PM · KMyMoney

Jan 2 2018

ostroffjh added a comment to D9592: Put all KMyMoney config files in kmymoney subdirectory.

I agree with the intent of this. However, if a user is testing both 4.8 and 5.x versions (assume with different kmy files) then once configuration files are moved by 5.x, will 4.8 create new default files, or will it find them in the new location?) I know it is an uncommon scenario, but we should avoid any surprises.

Jan 2 2018, 4:12 PM · KMyMoney

Jan 1 2018

ostroffjh added a comment to D9584: Fix automatic libofx detection.

What was the final decision regarding default behavior if OFX is not mentioned at all explicitly in the cmake command? I am OK with either behavior, but want to be sure it is what is intended.

Jan 1 2018, 10:46 PM
ostroffjh accepted D9586: Make sure to build widgets before any dialog.

Yes, this works for me also. It is still compiling, but it is far past where it failed before.

Jan 1 2018, 10:01 PM

Dec 31 2017

ostroffjh accepted D9561: Don't use memo as payee for investment transactions.

This does what it is supposed to. Ship it.

Dec 31 2017, 3:58 PM

Dec 29 2017

ostroffjh added a comment to D8267: Correcting symlinks between col- and row-resize and split_v and _h cursors.

Is there a problem with this, or has it simply fallen through the cracks?

Dec 29 2017, 10:11 PM · Plasma

Oct 17 2017

ostroffjh added a comment to D8267: Correcting symlinks between col- and row-resize and split_v and _h cursors.

Doing 'grep -ir "-resize" *' in the root of the repository returns no hits, I also looked at the results of grepping for resize and for split, and didn't see anything relevant.

Oct 17 2017, 3:55 PM · Plasma

Oct 16 2017

ostroffjh added a comment to D8263: Mention the share menu in the documentation.

Thanks again. I understand the danger of using jargon, but there are lots of descriptions of programs and menu items out there which are correct, but completely useless to the user who doesn't already know. Hopefully, by the time this actually hits distros, a user would be able to search and find info on KPurpose, or perhaps this could even (eventually, not necessary now) be turned into an appropriate link.

Oct 16 2017, 4:13 PM · Okular

Oct 14 2017

ostroffjh added a comment to D8263: Mention the share menu in the documentation.

ervin: Thanks. Would it then be worth mentioning KPurpose plugins instead of just "system settings?" I understand that text should stay concise, but if a user is trying to find out what that menu item does, the description should point to where it can be controlled. For me, "system settings" is too nebulous. Even if it means the "System Settings" app, it would help to say which section within it is relevant.

Oct 14 2017, 3:51 PM · Okular

Oct 13 2017

ostroffjh added a comment to D8263: Mention the share menu in the documentation.

I don't see this entry at all in my version of Okular 1.2.2 (17.08.2). Is it new, or does it's presence depend on whether there would be any entries?

Oct 13 2017, 4:50 PM · Okular

Oct 12 2017

ostroffjh created D8267: Correcting symlinks between col- and row-resize and split_v and _h cursors.
Oct 12 2017, 4:11 PM · Plasma

Oct 1 2017

ostroffjh created D8096: Added two most recent versions of Quicken for Windows for identity for OFX direct connect. Fixes Bug 385274.
Oct 1 2017, 8:11 PM

Aug 31 2017

ostroffjh committed R261:300047bcac08: Merge branch '4.8' of git.kde.org:kmymoney into 4.8 (authored by ostroffjh).
Merge branch '4.8' of git.kde.org:kmymoney into 4.8
Aug 31 2017, 7:36 PM
ostroffjh committed R261:06eec27a6885: REVIEW:130210 two more updates for whatsnew doc for 4.8.1 (authored by ostroffjh).
REVIEW:130210 two more updates for whatsnew doc for 4.8.1
Aug 31 2017, 7:36 PM
ostroffjh committed R261:2ea42b731a5f: Final doc changes for 4.8.1 release. (authored by ostroffjh).
Final doc changes for 4.8.1 release.
Aug 31 2017, 7:36 PM
ostroffjh committed R261:f24cbd9fcb1d: Merge branch '4.8' of git.kde.org:kmymoney into 4.8 (authored by ostroffjh).
Merge branch '4.8' of git.kde.org:kmymoney into 4.8
Aug 31 2017, 7:36 PM
ostroffjh committed R261:1f45fa59c0c9: accept Thomas' suggested edits (authored by ostroffjh).
accept Thomas' suggested edits
Aug 31 2017, 7:36 PM
ostroffjh committed R261:0854a68b4f25: initial doc update for 4.8.1 (authored by ostroffjh).
initial doc update for 4.8.1
Aug 31 2017, 7:36 PM

Jun 26 2017

ostroffjh added a comment to D6384: Switch to c++14.

Even if that would suffice, it would only be for OpenSuse. Every distro goes at their own pace for such upgrades, and LTS versions often are quite behind the leading edge.

Jun 26 2017, 4:22 PM · KMyMoney

Apr 29 2017

ostroffjh added a comment to D5655: Remove the backup function.

I agree that this functionality should be maintained It might not be necessary if a user has an overall backup system set up, and I have thought about using git to maintain a full backup history (but would probably be more useful for the uncompressed xml file). While it may take a few mouse clicks, I think it is fewer than launching a separate program (which would need initial configuration) and the addition of the timestamp to the filename is extremely useful. Also, I do not use the "mount" option, but I can see it being useful to keep the backups on an external thumb drive for example. Perhaps it would be worth asking on the user list whether folks find it useful?

Apr 29 2017, 5:22 PM · KMyMoney

Feb 17 2017

ostroffjh added a comment to D4655: Add range tab to chart reports.

Christian - I think the reason is in the original bug "... changes in e.g. investments with large different values are hardly to recognize on a linear scale (e.g. change of 20 to 10 if you have a graph changing from 150 to 70)." I'd probably like to see some actual examples of cases where it makes sense, but I don't see any reason against doing it.

Feb 17 2017, 11:03 PM · KMyMoney

Nov 6 2016

ostroffjh added a watcher for KMyMoney: ostroffjh.
Nov 6 2016, 5:21 PM
ostroffjh added a member for KMyMoney: ostroffjh.
Nov 6 2016, 5:21 PM