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.

Sign up for GitHub

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

Jump to bottom

chore: remove adpr env #2282

Merged
merged 2 commits into from
Oct 9, 2023
Merged

chore: remove adpr env #2282

merged 2 commits into from
Oct 9, 2023

Conversation

nguyenhoaidanh
Copy link
Contributor

@nguyenhoaidanh nguyenhoaidanh commented Oct 7, 2023 •
edited
Loading

  • if you want to switch env: => ➡️ change this run: yarn build-prod in pr.yaml
  • if you want to use dev env and some API prod env: ➡️ run: yarn build-dev and update API in .env.dev
  • of course, don't forget to revert all of them before merge

@kyber-ci-bot
Copy link

Auto Deploy Pull Request

Your domain kyberswap-interface-2282.pr.kyberengineering.io has been created.

Notes

  • Please wait up to 60s, we need to pull your image.
  • This domain only validity until this PR is closed

namgold
namgold reviewed Oct 9, 2023
View reviewed changes
src/utils/keyframes.ts Outdated Show resolved Hide resolved
@nguyenhoaidanh nguyenhoaidanh merged commit d54db26 into main Oct 9, 2023
6 checks passed
@nguyenhoaidanh nguyenhoaidanh deleted the refactor-env branch October 9, 2023 08:47
@kyber-ci-bot
Copy link

Auto Deploy Pull Request

Your domain kyberswap-interface-2282.pr.kyberengineering.io has been revoked because: PR closed.

Notes

  • Please re-open the PR to re-create this domain!

namgold pushed a commit that referenced this pull request Oct 11, 2023
@nguyenhoaidanh @namgold
chore: remove adpr env (#2282)
1b72f76
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@namgold namgold namgold approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nguyenhoaidanh @kyber-ci-bot @namgold