RE1-T88 Fixing removing Firebase from dockerfiles

[ucswift]

Summary by CodeRabbit

• Chores
  • Optimized container build configuration by removing unused dependencies from Docker build processes.

[request-info]

Thanks for opening this, but we'd appreciate a little more information. Could you update it with more details?

[coderabbitai]

Walkthrough

Removed COPY instructions for the Firebase provider csproj from three Dockerfiles, eliminating the Firebase provider restoration and build phase during image construction across multiple services.

Changes

Cohort / File(s)	Summary
Remove Firebase provider from Dockerfile build stages Web/Resgrid.Web.Services/Dockerfile, Web/Resgrid.Web/Dockerfile, Workers/Resgrid.Workers.Console/Dockerfile	Removed COPY instruction for Providers/Resgrid.Providers.Firebase/Resgrid.Providers.Firebase.csproj, excluding Firebase provider from project restoration and build context

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Verify that Firebase provider dependency is no longer required at runtime across all three services
• Confirm no downstream references to Firebase provider exist in the codebase after this change

Possibly related PRs

• RE1-T88 Firebase cleanup and updating Novu values sent #262: Removes the Firebase provider project entirely and its references, complementing this PR's exclusion of Firebase from Dockerfile build stages.

Poem

🐰 Firebase flies away, from Docker's keen eye,
Three images now lighter, we bid you goodbye!
No more restoration, no build-time delay—
The rabbit hops faster... without you today! 🔥

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly indicates the main change: removing Firebase from dockerfiles, which aligns with the changeset across all three Dockerfile modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch develop

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 52df831 and 232eb15.

📒 Files selected for processing (3)

• Web/Resgrid.Web.Services/Dockerfile (0 hunks)
• Web/Resgrid.Web/Dockerfile (0 hunks)
• Workers/Resgrid.Workers.Console/Dockerfile (0 hunks)

💤 Files with no reviewable changes (3)

• Web/Resgrid.Web.Services/Dockerfile
• Web/Resgrid.Web/Dockerfile
• Workers/Resgrid.Workers.Console/Dockerfile

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)

• GitHub Check: build
• GitHub Check: Analyze (csharp)
• GitHub Check: Analyze (javascript-typescript)
• GitHub Check: Analyze (javascript-typescript)
• GitHub Check: Analyze (csharp)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[ucswift]

Approve

[github-actions]

This PR is approved.