They are, this patch just provides the ability to save this data into the existing database using existing variables. This variables are the ones that are used in the print checks plugin.

In T11403#218245, @ostroffjh wrote:

Much better. Would it be worth the effort to add mention to the home page of the latest released version? If not on the Home page, at least on the Download page? I know I sometimes do go to an application's page just to confirm I'm actually on the latest released version. Right now, other than the News page, I don't see it anywhere within two or three clicks of the Home page.

Much better. Would it be worth the effort to add mention to the home page of the latest released version? If not on the Home page, at least on the Download page? I know I sometimes do go to an application's page just to confirm I'm actually on the latest released version. Right now, other than the News page, I don't see it anywhere within two or three clicks of the Home page.

I just moved the news to /news and increased the list of news per page to 10

Unfortunately, the number of items per page is fix for all pages in jekyll. Since the announcements on the first page are somewhat indistinguishable from the rest, how about having a News section between KMyMoney and Support in the top header. Creating a news subdir allows us to have another index.html (the pagination only works on index.html) but I have not tested that at all, just ideas

Yes, especially after the first page. Even the first page is larger than most screens, so putting 5 or even 10 items per page after the first seems OK. (It seems the announcements are clipped to at most a few lines, so they will never be very long.)

I just pushed a fix :) Maybe we could also increase the number of announcements displayed per page?

Thanks for all the work on this, but there are 12 pages, and the only thing that changes is Announcements, and even that took me a while to notice, since that section is not set off from the rest in any way. Also, the "Major Platform" icons are missing on all except the first page. The second is likely to be a bug somewhere. The first is more a style issue - so where is the best place to discuss how it "should" work, and what options there are.
Thanks.

kmymoney.org now use the new design and is deployed 🎉 Many thanks to all the peoples who helped with the update.

A nice blog-post about this task contains some guidance to setup the development environment in case that helps.

Hi @ognarb . I want to work on this website redesign during the season of KDE this time. Please guide me regarding this.

@adrianchavesfernandez Please request push access :) In the meantime Phabricator diff are also ok

Should I request push access until we finish the migration, or should I use Phabricator differentials?

I already added a .htaccess to the jekyll branch with some redirection so that the user don't get 404 errors.

Keeping existing URLs might be interesting in one or the other case, probably not all of them.

Should we use https://github.com/jekyll/jekyll-redirect-from to avoid breaking existing URLs, or it does not matter?

I'm sure we can work together :) I'm currently doing the support page (documentation, faq, ...), if you want you can do the homepage ;)

@ognarb I see you started already. Do you want help with this one, or should I pick a different website to port? (I’m not sure how fleasible parallel work on this is)

Ping?

I have not yet checked, but this would be a good time to be sure all the available Payee fields in the UI match those available in the check-printing plugin.

See D24233 for the implementation work

I will fix this

Corrected variable name numSplits.

Please change NumSplits into numSplits before you commit. Or do you want me do add it to the repository?

Changed all sahres to value and updated currency to print the currency symbol.

Added the currency for the accout=nt/transaction

questions

This should now give the proper amount.

I have replaced shares with value in: checkHTML.replace( valueVariable, MyMoneyUtils::formatMoney((*it).transaction().splits()[i].value().abs(), currency));.
As has been pointed out if one or more of the splits are in a different currency shares will give the wrong amount for the split.

Thanks for the review, please look at my changes.

Thanks for the feedback, my C++ skills are way out of date. I have made two of the three changes that were suggested. I do not understand the one about the currency and you reference line 206. That line gives the value in text not numbers. Line 207 gives the value in numbers and should be modified by the currency. My line 214 should give the value in numbers and is identical to line 207. It should be modified by the currency. If I am wrong please let me know.

Please review

Looks OK to me.

This looks fine to me.

This is now confirmed as fixed.

therefore it would be good to have context in the i18n() calls

According to https://techbase.kde.org/Development/Tutorials/Localization/i18n#Standard_Context_For_Common_Phrases you are refering to

https://dict.leo.org/englisch-deutsch/shares says:
shares Pl. (Brit.) [FINAN.] die Aktien Pl. [Börse]
shares Pl. [FINAN.] die Anteile Pl.
therefore it would be good to have context in the i18n() calls
otherwise +1

Yes, it is german only. I should have made the diff from the whole path.

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.