Skip to content

[TfL] Treat multiple location licences the same.#6003

Open
dracos wants to merge 1 commit into
masterfrom
fd-7071-required-bits-also
Open

[TfL] Treat multiple location licences the same.#6003
dracos wants to merge 1 commit into
masterfrom
fd-7071-required-bits-also

Conversation

@dracos

@dracos dracos commented Jun 1, 2026

Copy link
Copy Markdown
Member

Like column attachments, banners and temporary traffic signs will now also lock the street name / borough.
Factor this out to a shared role. [skip changelog] FD-7071

@dracos dracos requested a review from neprune June 1, 2026 15:49
@codecov

codecov Bot commented Jun 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.40%. Comparing base (d7b7986) to head (ea5caf5).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6003   +/-   ##
=======================================
  Coverage   83.40%   83.40%           
=======================================
  Files         500      501    +1     
  Lines       37306    37309    +3     
  Branches     6163     6163           
=======================================
+ Hits        31115    31118    +3     
- Misses       4379     4380    +1     
+ Partials     1812     1811    -1     

☔ 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.

Like column attachments, banners and temporary traffic
signs will now also lock the street name / borough.

Factor this out to a shared role.
@dracos dracos force-pushed the fd-7071-required-bits-also branch from 332ff9e to ea5caf5 Compare June 1, 2026 16:15

@neprune neprune left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants