Skip to content

Connect smart contract#32

Open
pravigya wants to merge 21 commits intofeat/connect-smart-contractsfrom
pravigya-connect-smart-contract
Open

Connect smart contract#32
pravigya wants to merge 21 commits intofeat/connect-smart-contractsfrom
pravigya-connect-smart-contract

Conversation

@pravigya
Copy link

No description provided.

@netlify
Copy link

netlify bot commented Nov 16, 2024

Deploy Preview for koryintia failed.

Name Link
🔨 Latest commit 3947e0f
🔍 Latest deploy log https://app.netlify.com/sites/koryintia/deploys/67388f4285d7f80008e28c9c

@Mrtn-Omwenga Mrtn-Omwenga changed the base branch from main to feat/connect-smart-contracts February 22, 2025 12:17
Mrtn-Omwenga and others added 20 commits March 12, 2025 17:41
- Improved loan ID handling to ensure correct type conversion
- Refactored collateral selection to improve maintainability
- Updated UI logic for better consistency
- Enhanced loan action logic with better state handling for 'Withdraw', 'Repay', 'Supply', and 'Liquidate'
- Improved health factor display logic
- Cleaned up UI components and added action-specific buttons for different loan statuses
- Refactored action types and modal interactions for better clarity and maintainability
- Replaced 'borrowedStatus' with 'loanStatus' in UpdateLoanDto and updated to use STATUS enum.
- Refactored ILoanRequest to use the STATUS enum for loanStatus instead of a string.
- Updated Home component to toggle & highlight the create loan panel based on screen size.
- Removed unused account & redux logic.
- Passed onToggleCreateLoan callback to Dashboard and Banner.
- Improved responsive behavior: mobile panel toggle and desktop highlight effect.
…e market loans filtering

    - Updated ActivePositionsTable to filter the incoming data so that only positions belonging to the current user (userAddress) are displayed.
    - Added the optional 'isMarket' prop in positions-cards to allow switching between market positions and user positions.
    - Updated getMarketLoans API call in positions-cards to filter only loans with statuses: requested, funded, and unhealthy.
…entifier handling

  - Updated update-loan API route to validate and convert loanId from params using parseInt
  - Changed MarketModal to call updateLoan with String(loanData.loanId) instead of loanData._id
  - Refactored getLoanNFTDetails to accept loanId as string and fundLoan to calculate required allowance
  - Ensured consistent use of loanId for contract interactions and API calls
…patibility

- Added @netlify/next adapter and configured netlify.toml for proper Next.js SSR support
- Fixed 'verifyMessage' import from ethers to avoid build-time type errors
- Ensured numeric types are casted with Number(...) for ethers.js compatibility
- Updated undefined/null-safe calls and added fallback values for better type handling
- Cleaned up deprecated or incorrect usages in UI components (e.g. missing keys, image props)
- Unified loanStatus enums using STATUS constant across model and API
- Ensured that route.ts APIs handle missing query parameters safely
- Applied minor refactors to improve maintainability and avoid runtime edge cases
  - Replaced raw ABI import in Oracle instantiation with correct .abi field usage
  - Updated src/abis/*.json files with latest build artifacts (Oracle, Treasury)
  - Fixed ethers.Contract instantiation to avoid InterfaceAbi type errors
  - Ensured ABI consistency across LoanPositionManager, Oracle, NFT, and Treasury
@netlify
Copy link

netlify bot commented Apr 4, 2025

Deploy Preview for koryintia ready!

Name Link
🔨 Latest commit 1a5428d
🔍 Latest deploy log https://app.netlify.com/sites/koryintia/deploys/67f036005f277f0008351c07
😎 Deploy Preview https://deploy-preview-32--koryintia.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@picaroloco picaroloco force-pushed the pravigya-connect-smart-contract branch from f545383 to 1a5428d Compare April 4, 2025 19:41
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.

3 participants