Skip to content

Conversation

@MantisClone
Copy link
Contributor

Purpose

BanklessDAO has reorganized and rebranded, fully independent from the Bankless brand. We want to redirect the old bankless.community domain to the new Black Flag DAO Notion Site to aid in the transition.

Changes Made

  • chore: redirect bankless.community and subdomains hosted by the BanklessDAO Vercel team to the Black Flag DAO Notion page.

Details

Considerations

Next Steps

Risks

  • Will BanklessHQ allow us to migrate the BanklessDAO website to https://banklessdao-website.vercel.app?: Bankless HQ has previously used Dopple to takedown websites impersonating the Bankless website and brand. They went so far as to accuse a developer preview of the tlBank page, hosted in Cloudflare, of phishing - resulting in @MantisClone's Cloudflare account getting suspended for almost 9 months. Presumably, they made an exception for the BanklessDAO website hosted at https://bankless.community. It's unclear how this exception works - does it whitelist an entire Vercel team, Cloudflare org, or individual domains? The exception seems to have included https://bankless-website-three.vercel.app which has existed for a long time.

@MantisClone MantisClone requested a review from a team as a code owner January 24, 2025 19:31
@MantisClone MantisClone requested a review from a user January 24, 2025 19:31
@MantisClone MantisClone self-assigned this Jan 24, 2025
@MantisClone MantisClone marked this pull request as draft January 24, 2025 19:33
@MantisClone
Copy link
Contributor Author

The pre-commit check failed because terraform is not installed in the CI environment.

Run pre-commit/action@v2.0.0
install pre-commit
/opt/hostedtoolcache/Python/3.12.8/x64/bin/pre-commit run --show-diff-on-failure --color=always --all-files
[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
[WARNING] repo `https://github.com/pre-commit/pre-commit-hooks` uses deprecated stage names (commit, push) which will be removed in a future version.  Hint: often `pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks` will fix this.  if it does not -- consider reporting an issue to that repo.
[INFO] Initializing environment for https://github.com/antonbabenko/pre-commit-terraform.
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
Fix End of Files.........................................................Passed
Check for added large files..............................................Passed
Terraform fmt............................................................Failed
- hook id: terraform_fmt
- exit code: 12[7](https://github.com/BanklessDAO/infrastructure/actions/runs/12956309822/job/36142299803?pr=40#step:6:8)

/home/runner/.cache/pre-commit/repo1zbt_8jz/terraform_fmt.sh: line 25: terraform: command not found
/home/runner/.cache/pre-commit/repo1zbt_8jz/terraform_fmt.sh: line 25: terraform: command not found
/home/runner/.cache/pre-commit/repo1zbt_8jz/terraform_fmt.sh: line 25: terraform: command not found
/home/runner/.cache/pre-commit/repo1zbt_8jz/terraform_fmt.sh: line 25: terraform: command not found

Error: The process '/opt/hostedtoolcache/Python/3.12.8/x64/bin/pre-commit' failed with exit code 1

@MantisClone MantisClone marked this pull request as ready for review January 24, 2025 19:43
cd $HOME
wget https://releases.hashicorp.com/terraform/${TERRAFORM_VERSION}/terraform_${TERRAFORM_VERSION}_linux_amd64.zip
unzip terraform_${TERRAFORM_VERSION}_linux_amd64.zip
sudo mv terraform /usr/local/bin/
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@ghost ghost merged commit 42e9b11 into BanklessDAO:main Jan 24, 2025
1 check passed
@MantisClone MantisClone deleted the redirect branch January 24, 2025 23:37
This pull request was closed.
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