Skip to content

Conversation

thejus03
Copy link
Contributor

@thejus03 thejus03 commented Jul 9, 2025

Context

Fixes #4095

Implementation

Minor bug in changing the key from uniqueKey to displayText for another line too. Previously in #4091 I only updated one line but there was another line that needed to be changed to fix this too.

Copy link

vercel bot commented Jul 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nusmods-export ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 9, 2025 6:24am

Copy link

vercel bot commented Jul 9, 2025

@thejus03 is attempting to deploy a commit to the modsbot's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

codecov bot commented Jul 9, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 52.88%. Comparing base (988c6fd) to head (e65a074).
⚠️ Report is 132 commits behind head on master.

Files with missing lines Patch % Lines
website/src/views/optimiser/OptimiserContent.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4096      +/-   ##
==========================================
- Coverage   54.52%   52.88%   -1.65%     
==========================================
  Files         274      287      +13     
  Lines        6076     6643     +567     
  Branches     1455     1622     +167     
==========================================
+ Hits         3313     3513     +200     
- Misses       2763     3130     +367     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@leslieyip02
Copy link
Member

Please hold on for a bit. I'm refactoring the optimiser functions into a utils file (#4094), so I've moved and modified the free day conflict function, so it should already be fixed in that version.

@thejus03
Copy link
Contributor Author

thejus03 commented Jul 9, 2025

sounds good!

@thejus03 thejus03 closed this Aug 13, 2025
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.

Optimiser free day conflict is not working properly
2 participants