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.

Sign up for GitHub

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

Jump to bottom

Fix duplicate patients with multiple visits in CSV upload #212

Merged
merged 4 commits into from
Jul 24, 2024
Merged

Fix duplicate patients with multiple visits in CSV upload #212

merged 4 commits into from
Jul 24, 2024

Commits on Jul 22, 2024

  1. Don't duplicate patients with multiple visits in CSV upload

    @mbarton
    mbarton committed Jul 22, 2024
    Configuration menu
    Copy the full SHA
    3922330 View commit details
    Browse the repository at this point in the history

  2. No need to look up the cohort for every row

    @mbarton
    mbarton committed Jul 22, 2024
    Configuration menu
    Copy the full SHA
    a58f51a View commit details
    Browse the repository at this point in the history

Commits on Jul 23, 2024

  1. Avoid duplicate patients by grouping by NHS number 

    As opposed to looking them up within the Audit Cohort. This method is compatible with the async work in #200. We're able to do this as CSV uploads always involve creating a new cohort.
    @mbarton
    mbarton committed Jul 23, 2024
    Configuration menu
    Copy the full SHA
    00bd579 View commit details
    Browse the repository at this point in the history

  2. Don't reorder rows

    @mbarton
    mbarton committed Jul 23, 2024
    Configuration menu
    Copy the full SHA
    700b9c0 View commit details
    Browse the repository at this point in the history