Skip to content

FT contributions: fix multiple exchanges from same key - #1586

Merged
mrvisscher merged 1 commit into
LCA-ActivityBrowser:mainfrom
marc-vdm:1585-FT-cont
Oct 16, 2025
Merged

mrvisscher merged 1 commit into
LCA-ActivityBrowser:mainfrom
marc-vdm:1585-FT-cont

Conversation

@marc-vdm

@marc-vdm marc-vdm commented Oct 13, 2025 •

Copy link
Copy Markdown
Member

problem was with multiple exchanges from same key, where only one would be counted. Now all exchanges to/from same key are summed correctly.

Checklist

If you have write access (otherwise a maintainer will do this for you):

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, feature, ui, change, documentation, breaking, ci
    as they show up in the changelog.
  • Add a milestone to the PR (and related issues, if any) for the intended release.
  • Request a review from another developer.

@marc-vdm marc-vdm added this to the 2.11.3 milestone Oct 13, 2025
@marc-vdm
marc-vdm requested a review from mrvisscher October 13, 2025 09:47
@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 53.153% (+0.01%) from 53.139%
when pulling f923da9 on marc-vdm:1585-FT-cont
into 5e093d0 on LCA-ActivityBrowser:main.

@mrvisscher
mrvisscher merged commit 2fdc2ac into LCA-ActivityBrowser:main Oct 16, 2025
10 checks passed
@marc-vdm
marc-vdm deleted the 1585-FT-cont branch October 21, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FT Contributions mis-counting double exchanges from same key

3 participants