Automatic import of classifications

Hey @Alfons1Qvor12 , you were right. Actually your description (“(the id of length 10 after the “?id=”, something like 0P00012345)”) led me to the assumption that we need the id attribute…

Now I tested with “normal” xml and it worked!

What I also noticed is that if I rename a classification it get’s re-created freshly when I run the script again. It would be nice to make this configurable as I would like to give them special names so that I see that they are the automatic classifications from your script/morningstar (I did it with a prefix “ac_”).

But so far - nice work! Thanks a ton!


@AndreasB , this test brought up another feature request towards PP itself: It would be nice if the classifications can be sorted or ordered via drag’n’drop. Don’t know if this is already possible somehow (like in some config files), but at least I don’t see the option in the GUI.
PS: An I don’t know where to put my feature requests… Is this the right way?

That is implemented. Have you tried? There is no “visible” option for drag’n’dop other than start dragging a classification or instrument in the tree views.

Maybe I explained it wrong. I don’t mean inside a classification. I mean the classifications themselves at the left panel. Can they be sorted?

Rechtscklick:

:man_facepalming:
ok, da hätte ich auch drauf kommen können. sorry.
aber sortieren und drag’n’drop wäre schon noch cooler. :wink:

Danke!

Du darfst gerne einen PR auf github einstellen.

haha. ich glaube, meinen Code wollt ihr nicht. Das ist Spaghetti code aus den 90gern…

oder meintest Du Feature Request, also macht Ihr die Anfragen auch über github?
Dann muss ich mir das mal ansehen.

BTW: Is there a tool/way to fill the Additional Attributes in PP (Logo, Gesamtkostenquote (TER), Fondsgröße, Anbieter, …)? In principle, some of them could also be retrieved from Morningstar.

I’m not familiar with any. That would be a nice one ;).

@Alfons1Qvor12 could you please check those

ETFs without holdings:
FUSA.DE / IE00BYXVGY31
IS3R.DE / IE00BP3QZ825

Wrong Region/Country stocks:
BABA / US01609W1027 (Asia/China expected)
GRAB / KYG4124C1096 (Asia expected)

+1, would be awesome

1 Like