Skip to content

Aichat - #17

Merged
nilushan merged 2 commits into
masterfrom
aichat
Nov 21, 2025
Merged

nilushan merged 2 commits into
masterfrom
aichat

Conversation

@nilushan

Copy link
Copy Markdown
Owner

No description provided.

nilushan and others added 2 commits November 21, 2025 19:50
Changed workflow to deploy hosting automatically while keeping functions
deployment manual due to IAM permissions requirement.

GitHub Actions will now:
- ✅ Auto-deploy hosting on push to master
- ⚙️  Functions require manual deployment: firebase deploy --only functions

To enable auto-deployment of functions, add "Service Account User" role to
the service account at:
https://console.cloud.google.com/iam-admin/iam?project=nilushandevelopment

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Added jsryudev/deploy-firebase-functions action to enable automatic
deployment of Cloud Functions alongside hosting.

Setup required:
1. Run locally: firebase login:ci
2. Add FIREBASE_TOKEN secret to GitHub repository

Workflow now:
- ✅ Deploys hosting via FirebaseExtended action (service account)
- ✅ Deploys functions via jsryudev action (FIREBASE_TOKEN)
- ✅ Prepares functions/dist with server bundle before deployment

This approach avoids IAM permission issues while enabling full CI/CD.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@nilushan
nilushan merged commit fb03bae into master Nov 21, 2025
1 check passed
@github-actions

Copy link
Copy Markdown

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

https://nilushansilva--pr17-aichat-bslhxxoh.web.app

(expires Fri, 28 Nov 2025 09:56:33 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f724968d440e28b28049b4ce218cbf77cd81c98c

@nilushan
nilushan deleted the aichat branch November 21, 2025 11:09
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.

1 participant