Skip to content

flutterfire-configure: add page #16831

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Jun 15, 2025
Merged

Conversation

Noibisjunior
Copy link
Contributor

This PR adds a new TLDR page for the flutterfire configure command, part of the FlutterFire CLI used to configure a Firebase project for a Flutter app.

  • The page is in the correct platform directory: common.
  • The page has at most 8 examples.
  • The page description has a link to documentation.
  • The page follows the content guidelines.
  • The page follows the style guide.
  • The PR title conforms to the recommended templates.
  • Version of the command being documented (if known): FlutterFire CLI v1.0.0-dev.54

@github-actions github-actions bot added new command Issues requesting creation of a new page or PRs adding a new page for a command. review needed Prioritized PRs marked for reviews from maintainers. labels Jun 11, 2025
@Managor
Copy link
Collaborator

Managor commented Jun 11, 2025

Remove the unrelated files.

@Noibisjunior
Copy link
Contributor Author

Remove the unrelated files.

Fixed and Deleted, Thanks

@tldr-bot
Copy link

The build for this PR failed with the following error(s):

pages/common/jsonTool.md:0: TLDR109 File name should be lowercase

Please fix the error(s) and push again.

Copy link
Collaborator

@Managor Managor left a comment

Choose a reason for hiding this comment

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

The unnecessary files are still there. Delete them.

@Noibisjunior Noibisjunior requested a review from Managor June 12, 2025 17:34
@Managor
Copy link
Collaborator

Managor commented Jun 12, 2025

Why did you ask for my review when you didn't address my request?

@Noibisjunior
Copy link
Contributor Author

The unnecessary files are still there. Delete them.

I have deleted them completely, please check.

Thanks

@Noibisjunior Noibisjunior requested review from Managor June 13, 2025 14:20
@Noibisjunior Noibisjunior requested a review from Managor June 13, 2025 17:03
Copy link
Member

@spageektti spageektti left a comment

Choose a reason for hiding this comment

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

Hi. Thanks for your contribution.

@spageektti
Copy link
Member

@Managor @sebastiaanspeck can you check if I added the file correctly? It was first time I was using git in termux

@Managor
Copy link
Collaborator

Managor commented Jun 14, 2025

Github claims that there is only one file changed so I assume you got it right

@Noibisjunior Noibisjunior requested a review from spageektti June 14, 2025 20:49
Copy link
Member

@MachiavelliII MachiavelliII left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution, and welcome to tldr!

@Managor Managor merged commit 4f9e037 into tldr-pages:main Jun 15, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new command Issues requesting creation of a new page or PRs adding a new page for a command. review needed Prioritized PRs marked for reviews from maintainers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants