Skip to content

Conversation

parlough
Copy link
Member

@parlough parlough commented Aug 2, 2025

Contributes to clean up portions of #12367.

@parlough parlough requested a review from ericwindmill August 2, 2025 07:05
@flutter-website-bot
Copy link
Collaborator

flutter-website-bot commented Aug 2, 2025

Visit the preview URL for this PR (updated for commit 0af16ab):

https://flutter-docs-prod--pr12261-feat-remove-codelab-cookboo-q4qyxa5w.web.app

@parlough parlough force-pushed the feat/remove-codelab-cookbook-index branch from 8ad613d to a26edeb Compare August 2, 2025 07:09
@parlough parlough marked this pull request as ready for review August 19, 2025 12:07
@parlough parlough requested review from sfshaza2, antfitch and a team as code owners August 19, 2025 12:07
@antfitch
Copy link
Contributor

@parlough do we have a project for this? If not, can you add one for this?

@parlough
Copy link
Member Author

@parlough do we have a project for this? If not, can you add one for this?

It's follow-up cleanup from a variety of Eric and other's projects, so I'm not sure if there's a specific project. I went ahead and created #12367 as an epic, consolidated issue for these pieces and added it to the Flutter docs 2025 project. I'll add some more details and links to other portions to that issue soon.

@parlough
Copy link
Member Author

@sfshaza2 Do you have any concerns with retiring these pages in favor of the combined learning resource index? If so, let me know how we can improve the index to get to a point where you'd be okay with this change. Thanks :D

Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

I dived in to the learning resources page. I don't like it as well as the pages we're replacing, but I do like that it will be easier to maintain. Having said that, I'm a bit surprised that using Flame is considered a good tutorial for beginners...

@parlough
Copy link
Member Author

parlough commented Sep 2, 2025

Thanks for taking a look @sfshaza2!

I don't like it as well as the pages we're replacing, but I do like that it will be easier to maintain.

That's definitely fair. They were a bit more focused. After the Jaspr migration, I'll explore further improvements to this page, but if you have any specific issues with or ideas for the index, do let me know.

Having said that, I'm a bit surprised that using Flame is considered a good tutorial for beginners...

I think that specific codelab is more about showing the possibilities, as it directly gives most of the code. We can definitely remove that tag though if you prefer!

@parlough parlough merged commit 1395030 into main Sep 2, 2025
9 checks passed
@parlough parlough deleted the feat/remove-codelab-cookbook-index branch September 2, 2025 15:06
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.

5 participants