Skip to content
This repository has been archived by the owner on Sep 13, 2024. It is now read-only.

feat: migrate rendering content on demand #125

Merged
merged 2 commits into from
Jul 8, 2024

Conversation

toph-allen
Copy link
Collaborator

In addition to basic migration:

  • Made some edits to code comments for formatting consistency.
  • Fixed some bugs in the code.

@toph-allen toph-allen requested a review from tdstein as a code owner July 3, 2024 20:40
@tdstein
Copy link
Collaborator

tdstein commented Jul 8, 2024

Do we need to include #71 in this change?

@toph-allen
Copy link
Collaborator Author

I didn't see that. I'll take a look in a sec and update this PR if so.

@nealrichardson
Copy link
Contributor

Yes, please include my changes from #71 if you're rewriting this recipe

@toph-allen toph-allen merged commit 3acf099 into main Jul 8, 2024
1 check passed
@toph-allen toph-allen deleted the toph/migrate-rendering-content-on-demand branch July 8, 2024 15:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants