Skip to content

[Feed] Incorporate Ration allowance#2740

Draft
matthew7838 wants to merge 11 commits intodevfrom
ration-allowance
Draft

[Feed] Incorporate Ration allowance#2740
matthew7838 wants to merge 11 commits intodevfrom
ration-allowance

Conversation

@matthew7838
Copy link
Collaborator

Context

Issue(s) closed by this pull request: closes #2578

What

Why

How

Test plan

Input Changes

Output Changes

  • N/A

Filter

@matthew7838 matthew7838 self-assigned this Feb 10, 2026
@github-actions
Copy link
Contributor

Current Coverage: %

Mypy errors on ration-allowance branch: 1686
Mypy errors on dev branch: 1685
1 more errors on ration-allowance branch

@github-actions
Copy link
Contributor

🚨 Please update the changelog. This PR cannot be merged until changelog.md is updated.
🚨 Flake8 linting errors were found. Please fix the linting issues.
🚨 Some tests have failed.

@JoeWaddell
Copy link
Collaborator

This is working as expected! Tested to see things pass/fail via different thresholds.

Note that I had to lift the maximums in the metadata properties file (thinking about it, given how tailored the maximum metadata value will be to a given farm it might just have to be removed, or just set to something like 500k).

@github-actions
Copy link
Contributor

Current Coverage: %

Mypy errors on ration-allowance branch: 1687
Mypy errors on dev branch: 1684
3 more errors on ration-allowance branch

@github-actions
Copy link
Contributor

🚨 Please update the changelog. This PR cannot be merged until changelog.md is updated.
🚨 Flake8 linting errors were found. Please fix the linting issues.
🚨 Some tests have failed.

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.

[Feed][FeedManager] Incorporate Ration Interval Purchase Allowance

2 participants