Skip to content

Conversation

@Lotendan
Copy link

@wahlbrink
Copy link
Contributor

Isn't this string a short form of "Date of last historical quote" (see following string in the file)? So the noun is "quote" ≙ "cotation" (f).

@buchen
Copy link
Member

buchen commented Nov 21, 2025

@Lotendan

The column shows the date of the latest historical price.

@Lotendan
Copy link
Author

I understand we are referring to a "cotation historique". But using the feminine together with "historique" alone simply does not work. "Cotation" as a noun is a feminine. "Historique" as a noun is a masculine.

In any case there is a discrepancy because we also have this string that is used in the same view (list of instruments):

ColumnDateFirstHistoricalQuote = Premier historique (Date)

So either we have:

Premier historique (Date) and Dernier historique (Date)
or
Première historique (Date) and Dernière historique (Date)
But not a mix of both ;) So in any case we need to fix something.

Otherwise how about we use "cotation" instead of "historique", which looks more natural to me?
Première cotation (Date) and Dernière cotation (Date)?

@buchen
Copy link
Member

buchen commented Nov 23, 2025

Thanks for pointing out the inconsistency. We should fix this.

Quick as background: because some APIs differentiate between "historical" prices (end of day prices) and the "latest" price (the current price which may or may have a delay while trading is ongoing), Portfolio Performance is storing prices also in two places: a list for the historical prices, and one price as the latest price.

  • ColumnDateFirstHistoricalQuote
  • ColumnLatestHistoricalDate

--> looks at the list of historical prices and returns the date of the first and last

  • ColumnLatestDate

--> looks at both the historical prices and the lastest prices and determines what is the most recent date (it could be the case that the API with the latest ist actually not returning any data so the historical prices are actually newer).

--> that is the reason why ColumnLatestHistoricalDate and ColumnLatestDate can differ

Long story short, that is the reason why also the labels expose some of this information.

Maybe the full label makes the most sense then (the user can abbreviate if he wants to) (I do not speak French):

Première cotation historique (Date)
Dernière cotation historique (Date)

@Lotendan Lotendan force-pushed the bugfix/DerniereHistorique branch from 3921047 to 17d4009 Compare November 24, 2025 20:02
@Lotendan
Copy link
Author

Thanks for the comments. Really helpful.
I have applied the change based on your inputs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants