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

Pro 2857 #153

Merged
merged 4 commits into from
Dec 10, 2024
Merged

Pro 2857 #153

merged 4 commits into from
Dec 10, 2024

Conversation

nikhilkumar1612
Copy link
Contributor

Description

  • added support for verifying paymaster.
  • 2 new endpoints to deploy paymaster and add stake.
  • db changes to store vp addresses (for both epv6 and epv7).

Types of changes

What types of changes does your code introduce?

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Other (please describe):

Further comments (optional)

please find the postman collection which has the api which is used for
2857.postman_collection.json

Copy link

linear bot commented Dec 6, 2024

tx = await contract.addStake("10", {value: ethers.utils.parseEther(amount), gasPrice: feeData.gasPrice});
} else {
tx = await contract.addStake(
"10",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

i have hardcoded the unstakeDelaySec param to be sent to smart contract as 10 @ch4r10t33r
let me know if this is something to be accepted from client side.

Copy link

cloudflare-workers-and-pages bot commented Dec 6, 2024

Deploying arka with  Cloudflare Pages  Cloudflare Pages

Latest commit: e0982bc
Status: ✅  Deploy successful!
Preview URL: https://ca2136b5.arka-3qg.pages.dev
Branch Preview URL: https://pro-2857.arka-3qg.pages.dev

View logs


async function up({ context: queryInterface }) {
await queryInterface.addColumn(
{schema: process.env.DATABASE_SCHEMA_NAME, tableName: 'arka_whitelist'},
Copy link
Member

Choose a reason for hiding this comment

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

Pls remove this additional column and let's retain a single endpoint for whitelist which is common for both EP6 and EP7.

Copy link
Member

@ch4r10t33r ch4r10t33r left a comment

Choose a reason for hiding this comment

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

LHTM. Pls lint the code (if not already done) and then merge.

@nikhilkumar1612 nikhilkumar1612 merged commit a36f6dd into master Dec 10, 2024
4 checks passed
@nikhilkumar1612 nikhilkumar1612 deleted the PRO-2857 branch December 10, 2024 21:02
nikhilkumar1612 added a commit that referenced this pull request Dec 11, 2024
This reverts commit a36f6dd.

reverting breaking changes for ep06 vp support
nikhilkumar1612 added a commit that referenced this pull request Dec 11, 2024
This reverts commit a36f6dd.

reverting breaking changes for ep06 vp support
nikhilkumar1612 added a commit that referenced this pull request Dec 25, 2024
* ft: adding support for verifying paymaster

* update: package.json version

* fix: changing names for mode, minor fix for entry point error message

* changes: common whitelist endpoints for v1/v2, removed v2 endpoints for whitelist
@nikhilkumar1612 nikhilkumar1612 mentioned this pull request Dec 25, 2024
8 tasks
nikhilkumar1612 added a commit that referenced this pull request Dec 26, 2024
* Pro 2857 (#153)

* ft: adding support for verifying paymaster

* update: package.json version

* fix: changing names for mode, minor fix for entry point error message

* changes: common whitelist endpoints for v1/v2, removed v2 endpoints for whitelist

* adding changelog for backend

* chnages: version change, minor response changes for metadata and add stake routes
vignesha22 pushed a commit that referenced this pull request Jan 6, 2025
* Pro 2857 (#153)

* ft: adding support for verifying paymaster

* update: package.json version

* fix: changing names for mode, minor fix for entry point error message

* changes: common whitelist endpoints for v1/v2, removed v2 endpoints for whitelist

* adding changelog for backend

* chnages: version change, minor response changes for metadata and add stake routes
vignesha22 added a commit that referenced this pull request Jan 13, 2025
* coingecko database addition

* ft: adding cron jobs to increase cache hit ratio for oracle data (#157)

* ft: adding cron jobs to increase cache hit ratio for oracle data

* update version

* changes to make native oracle calls only for chainlink

* Fix/arb token (#158)

* fix for arb token in multi token paymaster

* update cron expression for erc20 oracle update cron

* comment and version update

* Pro 2857 (#153) (#159)

* Pro 2857 (#153)

* ft: adding support for verifying paymaster

* update: package.json version

* fix: changing names for mode, minor fix for entry point error message

* changes: common whitelist endpoints for v1/v2, removed v2 endpoints for whitelist

* adding changelog for backend

* chnages: version change, minor response changes for metadata and add stake routes

* coingecko database addition

* merge bug fixes

* updated version

* added required env vars to demo env

* changed to pro key

* changes done as requested

* changes to retain v2 endpoints, minor changes for change log

* updated changelog

---------

Co-authored-by: Nikhil Kumar <[email protected]>
Co-authored-by: nikhil kumar <[email protected]>
vignesha22 added a commit that referenced this pull request Jan 24, 2025
* coingecko database addition

* ft: adding cron jobs to increase cache hit ratio for oracle data (#157)

* ft: adding cron jobs to increase cache hit ratio for oracle data

* update version

* changes to make native oracle calls only for chainlink

* Fix/arb token (#158)

* fix for arb token in multi token paymaster

* update cron expression for erc20 oracle update cron

* comment and version update

* Pro 2857 (#153) (#159)

* Pro 2857 (#153)

* ft: adding support for verifying paymaster

* update: package.json version

* fix: changing names for mode, minor fix for entry point error message

* changes: common whitelist endpoints for v1/v2, removed v2 endpoints for whitelist

* adding changelog for backend

* chnages: version change, minor response changes for metadata and add stake routes

* coingecko database addition

* merge bug fixes

* updated version

* added required env vars to demo env

* changed to pro key

* changes done as requested

* changes to retain v2 endpoints, minor changes for change log

* updated changelog

---------

Co-authored-by: Nikhil Kumar <[email protected]>
Co-authored-by: nikhil kumar <[email protected]>
vignesha22 added a commit that referenced this pull request Jan 24, 2025
* coingecko database addition

* ft: adding cron jobs to increase cache hit ratio for oracle data (#157)

* ft: adding cron jobs to increase cache hit ratio for oracle data

* update version

* changes to make native oracle calls only for chainlink

* Fix/arb token (#158)

* fix for arb token in multi token paymaster

* update cron expression for erc20 oracle update cron

* comment and version update

* Pro 2857 (#153) (#159)

* Pro 2857 (#153)

* ft: adding support for verifying paymaster

* update: package.json version

* fix: changing names for mode, minor fix for entry point error message

* changes: common whitelist endpoints for v1/v2, removed v2 endpoints for whitelist

* adding changelog for backend

* chnages: version change, minor response changes for metadata and add stake routes

* coingecko database addition

* merge bug fixes

* updated version

* added required env vars to demo env

* changed to pro key

* changes done as requested

* changes to retain v2 endpoints, minor changes for change log

* updated changelog

---------

Co-authored-by: Nikhil Kumar <[email protected]>
Co-authored-by: nikhil kumar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants