Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Interactive Brokers Tiered pricing #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions data/brokers.json
Original file line number Diff line number Diff line change
Expand Up @@ -168,5 +168,23 @@
"costOverview": "https://www.interactivebrokers.ie/en/index.php?f=48059",
"website": "https://www.interactivebrokers.ie/en/home.php",
"affiliateLink": "https://ibkr.com/referral/nic106"
},
{
"name": "Interactive Brokers Tiered",
"logo": "interactive-brokers.png",
"product": "Tiered Pricing",
"etfTransactionFee": {
"percentage": 0.0558,
Copy link

@gerbenvl gerbenvl Jan 25, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Misschien wel even vastleggen hoe je aan deze 0,0558% komt. Want dit is je eigen benadering zodat het in dit model past.

Als ik even snel kijk dan klopt dit best redelijk. Alleen voor de inleg rond de 2500-6500 zit je ietwat aan de lage kant. Maar goed, dat gaat even niet anders in deze setup.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

JSON staat officieel volgens mij geen commentaar toe in het formaat, of is de door jullie gebruikte parser daar wat flexibeler in? Hoe/waar stel je anders voor dit te documenteren?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

JSON staat officieel volgens mij geen commentaar toe in het formaat, of is de door jullie gebruikte parser daar wat flexibeler in? Hoe/waar stel je anders voor dit te documenteren?

Die zou ik ff met @nicwortel checken, anders een losse readme.md ergens er bij?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

JSON ondersteunt inderdaad geen native comments, maar ik ben het met @gerbenvl eens dat het wenselijk is om hier wel wat uitleg bij te geven. Met name zodat in de toekomst makkelijk te herleiden is of de gebruikte tarieven nog up-to-date zijn of bijgewerkt moeten worden.

Wat wel kan is het toevoegen van extra properties aan de JSON, die niet uitgelezen worden door de code maar wel wat context kunnen geven.

Technisch gezien kan zo'n property elke willekeurige key/naam hebben die nog niet in gebruik is, maar om duidelijk te maken dat het om een comment gaat die geen effect heeft op de werking van de website zou het mijn voorkeur hebben dat zo'n property een naam als _comment of iets dergelijks heeft.

Bijvoorbeeld:

Suggested change
"percentage": 0.0558,
"_comment": "This is the sum of X, Y and Z",
"percentage": 0.0558,

"minimum": 1.89,
"maximum": 77.04
},
"mutualFundTransactionFee": {
"percentage": 0.1,
"minimum": 3.65,
"maximum": 74.10
},
"costOverview": "https://www.interactivebrokers.ie/en/index.php?f=48059",
"website": "https://www.interactivebrokers.ie/en/home.php",
"affiliateLink": "https://ibkr.com/referral/nic106"
}
]
5 changes: 5 additions & 0 deletions data/portfolios.json
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@
"DEGIRO",
"Saxo Bank",
"Interactive Brokers",
"Interactive Brokers Tiered",
"Lynx"
]
},
Expand All @@ -63,6 +64,7 @@
"brokers": [
"DEGIRO",
"Interactive Brokers",
"Interactive Brokers Tiered",
"Lynx",
"Saxo Bank"
]
Expand Down Expand Up @@ -107,6 +109,7 @@
"brokers": [
"Saxo Bank",
"Interactive Brokers",
"Interactive Brokers Tiered",
"Lynx"
]
},
Expand All @@ -128,6 +131,7 @@
"brokers": [
"Saxo Bank",
"Interactive Brokers",
"Interactive Brokers Tiered",
"Lynx"
]
},
Expand All @@ -145,6 +149,7 @@
"brokers": [
"DEGIRO",
"Interactive Brokers",
"Interactive Brokers Tiered",
"Lynx",
"Saxo Bank"
]
Expand Down
12 changes: 12 additions & 0 deletions wijzigingen.html
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,18 @@

<h1 class="h2">Wijzigingen</h1>

<article>
<h2 class="h4 mb-0">Kostenschema Interactive Brokers - Tiered Pricing</h2>

<p class="text-muted">24 januari 2021</p>

<p>Het alternatieve kostenschema voor Interactive Brokers (Tiered Pricing) is nu ook als aanbieder toegevoegd.
De kostenberekeningen gaan uit van verhandeling op de Duitse Xetra beurs, daarnaast vallen de werkelijke
kosten wat lager uit bij stortingen > 50.000. Er is dus wat afwijking mogelijk met de werkelijke kosten,
maar het biedt een aardige indicatie van de kosten in de meeste gevallen. Veelal komt dit ook in
werkelijkheid gunstiger uit dan een Fixed Pricing optie via Interactive Brokers.</p>
</article>

<article>
<h2 class="h4 mb-0">Beleggingsfondsen bij Interactive Brokers</h2>

Expand Down