Skip to content

Conversation

@penx
Copy link
Contributor

@penx penx commented Oct 31, 2022

Reopening #5, remix-run/remix#3398 against remix-run/examples

Deploy to firebase functions and hosting

Closes: Discussion at remix-run/remix#1811 (comment)

  • Docs
  • Tests (I'd happily add Playwright tests if you want these on examples)

Testing Strategy:

  • Testing locally against emulator with JS enabled and disabled
  • Testing locally against production with JS enabled and disabled
  • Following deploy instructions and testing remote with JS enabled and disabled

@MichaelDeBoey MichaelDeBoey changed the title feat(examples/firebase): deploy to Cloud Functions for Firebase feat(firebase): deploy to Cloud Functions for Firebase Nov 2, 2022
@github-actions github-actions bot added the needs-response We need a response from the original author about this issue/PR label Jan 1, 2023
@penx
Copy link
Contributor Author

penx commented Jan 20, 2023

@machour @mcansh @MichaelDeBoey

Similar to my comment on #52 (comment)

Please let me know if I should keep rebasing this or abandon it

@github-actions github-actions bot removed the needs-response We need a response from the original author about this issue/PR label Jan 20, 2023
@MichaelDeBoey
Copy link
Member

@penx Please rebase this branch onto latest main & resolve conflicts

@penx penx force-pushed the firebase-functions branch from dec444c to 56f86ed Compare January 24, 2023 22:55
@MichaelDeBoey MichaelDeBoey merged commit 56d77ba into remix-run:main Jan 25, 2023
@penx
Copy link
Contributor Author

penx commented Jan 25, 2023

@MichaelDeBoey thanks for merging, I was trying to figure out the yarn lock, which will now be out of date in main.

Hopefully will get there shortly!

@penx
Copy link
Contributor Author

penx commented Jan 25, 2023

I've resolved my issue with yarn, and the root lock file seems to be fine as it doesn't contain dependencies for each example

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.

3 participants