Skip to content

Inidivudal points#38

Merged
JEK58 merged 2 commits intomainfrom
inidivudal-points
Mar 6, 2026
Merged

Inidivudal points#38
JEK58 merged 2 commits intomainfrom
inidivudal-points

Conversation

@JEK58
Copy link
Copy Markdown
Owner

@JEK58 JEK58 commented Mar 6, 2026

No description provided.

@JEK58 JEK58 merged commit 5f48e6d into main Mar 6, 2026
1 check failed
@JEK58 JEK58 deleted the inidivudal-points branch March 6, 2026 12:12
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8875132e50

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +205 to +207
confirmed: simulation.confirmed ?? data.confirmed,
nationalities: simulation.nationalities ?? data.nationalities,
genders: simulation.genders ?? data.genders,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Stop falling back to baseline on empty simulation results

When the simulated selection has fewer than two ranked pilots (for example after clicking Clear all or leaving only one ranked pilot), the API returns confirmed/nationalities/genders as undefined, but this code immediately falls back to data.*, so the UI keeps showing the original forecast and charts instead of reflecting the empty/insufficient selection. This produces incorrect WPRS output for a reachable user flow and makes the interactive controls appear ineffective.

Useful? React with 👍 / 👎.

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.

1 participant