Skip to content

Conversation

@pregnantboy
Copy link
Contributor

@pregnantboy pregnantboy commented Oct 27, 2025

changes

  1. Moving UI components into a dedicated components directory
  2. Extracting step display logic into a new StepsToDisplayContext
  3. Replacing the StepExecutionsToIncludeContext with the more comprehensive StepsToDisplayContext
  4. Simplifying the MrfContextProvider by removing the explicit steps prop
  5. Creating a dedicated StepsList component to handle the rendering of steps (Editor component was getting bloated)

Copy link
Contributor Author

pregnantboy commented Oct 27, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch from 021e5c4 to 4215c69 Compare October 27, 2025 10:44
@pregnantboy pregnantboy force-pushed the mrf/refactor-create-step branch from c2ba285 to c1aa32f Compare October 27, 2025 10:44
@pregnantboy pregnantboy force-pushed the mrf/refactor-create-step branch from c1aa32f to 7758150 Compare November 26, 2025 09:20
@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch from 4215c69 to cc22c20 Compare November 26, 2025 09:20
@pregnantboy pregnantboy force-pushed the mrf/refactor-create-step branch from 7758150 to 91123ae Compare November 29, 2025 16:30
@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch 2 times, most recently from fc30efa to adc259d Compare November 30, 2025 08:19
@pregnantboy pregnantboy force-pushed the mrf/refactor-create-step branch from 91123ae to 5578bf8 Compare November 30, 2025 08:19
@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch from adc259d to fccf40b Compare November 30, 2025 08:56
@pregnantboy pregnantboy force-pushed the mrf/refactor-create-step branch from 5578bf8 to 36e484e Compare November 30, 2025 09:19
@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch from fccf40b to 0c41ea2 Compare November 30, 2025 09:19
@pregnantboy pregnantboy changed the title refactor(frontend): refactor contexts [MRF-13]: (frontend) refactor contexts Nov 30, 2025
@pregnantboy pregnantboy changed the title [MRF-13]: (frontend) refactor contexts [MRF-13] PLU-520: (frontend) refactor contexts Nov 30, 2025
@linear
Copy link

linear bot commented Nov 30, 2025

PLU-520 MRF: Approvals

@pregnantboy pregnantboy force-pushed the mrf/refactor-create-step branch from 36e484e to de041df Compare November 30, 2025 09:44
@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch from 0c41ea2 to 2482828 Compare November 30, 2025 09:44
@pregnantboy pregnantboy changed the title [MRF-13] PLU-520: (frontend) refactor contexts [MRF-13.5] PLU-520: (frontend) refactor contexts Nov 30, 2025
@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch from 2482828 to de2b674 Compare December 3, 2025 10:55
@pregnantboy pregnantboy force-pushed the mrf/refactor-create-step branch from de041df to aef16f2 Compare December 3, 2025 10:55
@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch from de2b674 to ad8d078 Compare December 3, 2025 15:42
@pregnantboy pregnantboy force-pushed the mrf/refactor-create-step branch 2 times, most recently from 6e177e5 to 86299bf Compare December 3, 2025 15:45
@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch from ad8d078 to 4404d93 Compare December 3, 2025 15:45
@datadog-opengovsg
Copy link

datadog-opengovsg bot commented Dec 3, 2025

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: e390d43 | Docs | Was this helpful? Give us feedback!

@pregnantboy pregnantboy force-pushed the mrf/refactor-contexts branch from 4404d93 to e390d43 Compare December 3, 2025 16:09
@pregnantboy pregnantboy force-pushed the mrf/refactor-create-step branch from 86299bf to b7e7415 Compare December 3, 2025 16:09
@pregnantboy pregnantboy marked this pull request as ready for review December 3, 2025 16:24
@pregnantboy pregnantboy requested a review from a team as a code owner December 3, 2025 16:24
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.

2 participants