Skip to content
This repository was archived by the owner on Aug 3, 2022. It is now read-only.

Adding CODEOWNERS#1

Open
ahrenstein wants to merge 1 commit into
mainfrom
CODEOWNERS
Open

Adding CODEOWNERS#1
ahrenstein wants to merge 1 commit into
mainfrom
CODEOWNERS

Conversation

@ahrenstein

Copy link
Copy Markdown

Please approve protecting the branch

@vercel

vercel Bot commented Oct 20, 2021

Copy link
Copy Markdown

@ahrenstein is attempting to deploy a commit to the BanklessDAO Team on Vercel.

To accomplish this, @ahrenstein needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

@SlinkyPotato

Copy link
Copy Markdown
Contributor

Hey @ahrenstein - I see in Vercel that your gmail account is listed as a member. I think you just need to connect that account with your github account

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

@ahrenstein

ahrenstein commented Oct 21, 2021 via email

Copy link
Copy Markdown
Author

@wpmonty

wpmonty commented Oct 22, 2021

Copy link
Copy Markdown
Member
If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

Seems like this might be pretty harsh to manage with our scale without everyone being members. How would a fork PR ever be merged from a non-L2?

SlinkyPotato pushed a commit that referenced this pull request Dec 16, 2021
Fix nft loading bug with missing axios await, and hydrate state.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants