Skip to content
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

chore: add supabase environment variables examples and readme #17

Conversation

gitcoindev
Copy link
Contributor

No description provided.

@0x4007
Copy link
Member

0x4007 commented Feb 3, 2024

Thanks for this but I can't merge because from what I understand these dependencies require Node APIs, which are not available in our Workers production environment.

@0x4007 0x4007 marked this pull request as draft February 3, 2024 07:03
@gitcoindev
Copy link
Contributor Author

Thanks for this but I can't merge because from what I understand these dependencies require Node APIs, which are not available in our Workers production environment.

I will try to read more on this over the weekend. Perhaps @whilefoo has also some ideas. I am quite sure we can solve it somehow.

@gentlementlegen
Copy link
Member

@0x4007 What is this sorcery haha

@gitcoindev
Copy link
Contributor Author

@0x4007 What is this sorcery haha

I do not remember exactly but I suspect the request for this was superseded by an another pull request, hence this one was closed -)

@gentlementlegen
Copy link
Member

Interesting to see that somehow it assigned me there just now with no activity here whatsoever

@0x4007
Copy link
Member

0x4007 commented Jun 18, 2024

Interesting to see that somehow it assigned me there just now with no activity here whatsoever

There is a transient bug where it would "cross contaminate" its execution. The invocation would be in one repo, and the result would be in another. It would maintain the pull/issue number, so you were likely working on some #17 on some repo @gentlementlegen, likely within the @ubiquity organization.

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