Skip to content
This repository has been archived by the owner on Oct 16, 2022. It is now read-only.

[FYI] Initial draft of a Sync from PP #65

Closed
buchen opened this issue Mar 4, 2021 · 1 comment
Closed

[FYI] Initial draft of a Sync from PP #65

buchen opened this issue Mar 4, 2021 · 1 comment
Labels
question Further information is requested

Comments

@buchen
Copy link

buchen commented Mar 4, 2021

I have a pushed a rudimentary sync to the branch feature_server_sync in buchen/portfolio. It is very drafty: PortfolioReportSync

Bildschirmfoto 2021-03-04 um 21 30 02

If you want to test it:

  • paste the session token into the preferences
  • choose Online -> Sync with portfolio-report from the menu

Things I noticed:

  • I was successfully able to create a portfolio and some securities, but I only could read some portfolios (the list did not include all) and never was able to read securities. And even for the portfolios, it took quite some time until the portfolios showed up after creation. Do you see the same behavior?
  • When creating a new security, I need to provide an empty string for wkn, ticker, note, etc. Is it possible to have these field nullable, i.e., I can leave them out?
@buchen buchen added the question Further information is requested label Mar 4, 2021
@tfabritius
Copy link
Owner

Thanks :)

The first issue you have noticed was probably a caching issue. Should be corrected by now.

I'll have a look at the empty/default values. Would you also leave out fields when sending an update? Should those fields that are missing be ignored in the update (i.e. not updated) or should they be set to an empty/non-existent value (like when creating)?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants