Skip to content

Conversation

@Akshit2004
Copy link

Added [firebase.js] a new Firebase helper module implementing configuration and basic helper functions.
Applied integration changes to [index.js] and related code to use the new Firebase helper.

Comment on lines 47 to 73

const packages = await checkbox({
let packages = await checkbox({
message: "Select optional packages:",
choices: [
{ name: "Axios", value: "axios" },
{ name: "React Icons", value: "react-icons" },
{ name: "React Hook Form", value: "react-hook-form" },
{ name: "Yup", value: "yup" },
{ name: "Formik", value: "formik" },
{ name: "Moment.js", value: "moment" }
{ name: "Moment.js", value: "moment" },
{ name: "Firebase (Firestore utils + env)", value: "firebase" }
]
});

// Fallback: some terminals or clients may not toggle checkboxes reliably.
// If Firebase wasn't picked above, ask a simple confirm so the user can't miss it.
if (!packages.includes('firebase')) {
try {
const firebaseConfirm = await confirm({ message: 'Would you like to add Firebase (Firestore utils + env)?', default: false });
if (firebaseConfirm) {
packages = [...packages, 'firebase'];
}
} catch (e) {
// ignore confirm errors and continue
}
}

Copy link
Owner

Choose a reason for hiding this comment

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

Create a dedicated confirmation command for firebase, which ask,
do you want to setup firebase in this project?

If yes then then just simply add it in packages and rest is setup.

@Akshit2004
Copy link
Author

Akshit2004 commented Sep 17, 2025

hey this is my first time contributing to open souce project, should i open a new pull request or is there way to implement here only.

also according your last message , env and basic boiler plate code for firebase wont be needed right?

@harshgupta20
Copy link
Owner

hey this is my first time contributing to open souce project, should i open a new pull request or is there way to implement here only.

also according your last message , env and basic boiler plate code for firebase wont be needed right?

You can follow the same way where isPwa is mentioned, since it is also the same kind of setup with optional configurations, something like isFirebase.

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

Labels

enhancement New feature or request New Feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Firebase support option with basic utility functions of CRUD and ENV variable setup

2 participants