feat: implement batch job fetching with get_jobs_batch contract function - #688
Open
TOS1ANO wants to merge 1 commit into
Open
feat: implement batch job fetching with get_jobs_batch contract function#688TOS1ANO wants to merge 1 commit into
TOS1ANO wants to merge 1 commit into
Conversation
Add get_jobs_batch(start, limit) public view function to smart contract returning Vec<Job>, skipping non-existent IDs safely. Add getJobsBatch() frontend helper. Replace NxgetJob(id) loops with single batch calls on Home, Dashboard, Transactions, Profile, Compare pages and disputes-loader. Fix JSX nesting bugs in layout.tsx and ToastProvider.tsx. Move SSR-disabled dynamic imports into ClientComponents.tsx.
|
@TOS1ANO Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
Owner
|
@TOS1ANO please resolve conflicts |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Linked Issue
Closes #587 [SC-101]
What Changed
Smart Contract
get_jobs_batch(start, limit)public view function returningVec<Job>Option<Job>patternget_job(id)kept unchangedFrontend
getJobsBatch(startId, limit)helper inlib/contract.tsgetJob(id)loops with single batch calls on:Bug Fixes
layout.tsx(swapped closing tags for TypographyProvider/NetworkProvider)</div>inToastProvider.tsxcausing JSX parse errorClientComponents.tsxto comply with Next.js App Router Server Component rulesValidation
Additional Notes