Skip to content

Conversation

roxdanila
Copy link
Contributor

@roxdanila roxdanila commented Jan 24, 2023

Context

Scripts required for UI mocks.
Closes #664
Depends on #639

Changes proposed in this pull request

Reimplementing existing scripts so deposits and buy covers work well.

Test plan

You can run locally:

  • ABI_DIR=../frontend-react/src/constants/abis CONFIG_FILE=../frontend-react/config/local.json node scripts/deploy/start.js
    Then:
  • HARDHAT_NETWORK=localhost CONFIG_FILE=$PWD/../frontend-react/config/local.json node scripts/mocks/fund.js 0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266
  • HARDHAT_NETWORK=localhost CONFIG_FILE=$PWD/../frontend-react/config/local.json node scripts/mocks/createStakingPools.js
  • HARDHAT_NETWORK=localhost CONFIG_FILE=$PWD/../frontend-react/config/local.json STAKER=0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266 node scripts/mocks/depositToStakingPools.js
  • HARDHAT_NETWORK=localhost CONFIG_FILE=$PWD/../frontend-react/config/local.json BUYER=0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266 node scripts/mocks/buyV2Covers.js

Checklist

  • Rebased the base branch
  • Attached corresponding Github issue
  • Prefixed the name with the type of change (i.e. feat, chore, test)
  • Performed a self-review of my own code
  • Followed the style guidelines of this project
  • Made corresponding changes to the documentation
  • Didn't generate new warnings
  • Didn't generate failures on existing tests
  • Added tests that prove my fix is effective or that my feature works

Review

When reviewing a PR, please indicate intention in comments using the following emojis:

  • 🍰 = Nice to have but not essential.
  • 💡 = Suggestion or a comment based on personal opinion
  • 🔨 = I believe this should be changed.
  • 🤔 = I don’t understand something, do you mind giving me more context?
  • 🚀 = Feedback

@roxdanila roxdanila force-pushed the feat/staking-viewer branch 2 times, most recently from f950375 to e3b6248 Compare January 24, 2023 11:31
@roxdanila roxdanila force-pushed the test/staking-scripts branch from f8b1430 to 0fb6479 Compare January 24, 2023 11:32
@roxdanila roxdanila force-pushed the feat/staking-viewer branch 2 times, most recently from d4e380c to cb1fc7b Compare January 26, 2023 12:39
@roxdanila roxdanila force-pushed the test/staking-scripts branch from 66157fb to 1ec797e Compare January 26, 2023 12:45
@shark0der shark0der force-pushed the feat/staking-viewer branch from cb1fc7b to 8fb5069 Compare January 26, 2023 12:46
@roxdanila roxdanila force-pushed the test/staking-scripts branch 4 times, most recently from 842f8b0 to 3f4ec6f Compare January 27, 2023 14:48
@roxdanila roxdanila requested review from mixplore, danoctavian and shark0der and removed request for mixplore and danoctavian January 30, 2023 10:19
@roxdanila roxdanila marked this pull request as ready for review January 30, 2023 10:19
@roxdanila roxdanila linked an issue Jan 30, 2023 that may be closed by this pull request
@roxdanila roxdanila force-pushed the test/staking-scripts branch from 6d1656a to 0e113a3 Compare February 1, 2023 13:16
@shark0der shark0der force-pushed the feat/staking-viewer branch from 1fdd9c9 to b711445 Compare February 3, 2023 08:29
@roxdanila roxdanila force-pushed the feat/staking-viewer branch from b711445 to 027e0a0 Compare February 3, 2023 11:50
@roxdanila roxdanila changed the base branch from feat/staking-viewer to feat/staking-viewer-temp February 3, 2023 11:53
@shark0der shark0der changed the base branch from feat/staking-viewer-temp to feat/staking-viewer February 6, 2023 12:41
@shark0der shark0der changed the base branch from feat/staking-viewer to nexus-v2 February 6, 2023 12:43
@shark0der shark0der force-pushed the test/staking-scripts branch from 02243ad to 0b0d48f Compare February 6, 2023 14:28
@shark0der shark0der merged commit fc14c89 into nexus-v2 Feb 6, 2023
@shark0der shark0der deleted the test/staking-scripts branch February 6, 2023 18:15
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.

Fix UI mocks scripts
3 participants