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.

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

Tracking pull request to merge release-1.0.0 to main #1852

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

kuanfandevops
Copy link
Collaborator

No description provided.

Copy link

github-actions bot commented Jan 29, 2025

Frontend Test Results

  1 files  125 suites   47s ⏱️
436 tests 416 ✅ 20 💤 0 ❌
438 runs  418 ✅ 20 💤 0 ❌

Results for commit 077d570.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 29, 2025

Backend Test Results

524 tests   524 ✅  1m 51s ⏱️
  1 suites    0 💤
  1 files      0 ❌

Results for commit 077d570.

♻️ This comment has been updated with latest results.

Alex Zorkin and others added 23 commits January 30, 2025 16:30
* Fuel code was using custom logic meaning it didnt get any of the yellow upgrades done previously
* Remove the FE logic to Backend validation and let the errors go through the normal validation route
* Method currently checks whichever ID was passed in, always check the from_organization_id for the balance.
* Remove txn:description text as per business request
* Previously a supplier would be blocked if they didnt have enough credits
* Now we take as many as we can and allow it
…1861

fix: Check from organization balance
…-1848

fix: Allow Submissions without enough credits
…-1809

Fix: LCFS - backend filter logic on the Compliance Report index #1809
Fix: Updated Materialized Views, Functions, Triggers - 1813
fix: Move Fuel Code validation to the back end
* It should be checking the summary penalty amount not the amounts in the report
Feat: Add Historical Rejected Status
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.

4 participants