Conversation
* Update README.md new logo * fix: fixed ci versions
* feat: update shares, refactor, basic test Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * fix: avoid trailing zero array entries Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: formatting and shortcut return if zero array Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: added some unit tests, batch updateShares and fixes. Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: updated tests Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: removed commented code Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: removed console logs Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: pr review changes Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * fix: pr feedback Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: formatting Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: updated tests, added requestsCountByUser mapping Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: max iteration check Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: formatting Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * fix: removed gas limit test as it broke the gas report Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * Feat: Depreciate request ids mapping (Sc-20631) (#31) * feat: updated tests to depreciate requestIds mapping (WIP) Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: fixed tests Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * fix: fixed returning wrong array elements Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> --------- Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * fix: pr review fixes Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * fix: pr review fixes Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: updated tests Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: naming conventions Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: Add TODOs and formatting * chore: Add TODOs and formatting * feat: refactored the view functions Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: renamed storage Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: added tests Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: cleanup * chore: cleanup --------- Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> Co-authored-by: 0xfarhaan <59924029+0xfarhaan@users.noreply.github.com>
* feat: fixed todo's Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: added multiple remove tests Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat:reordered some functions Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: fix formatting --------- Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> Co-authored-by: 0xfarhaan <59924029+0xfarhaan@users.noreply.github.com>
* feat: added userRequestSummaries Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * fix: missing share changes Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * fix: missing natspec Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: added tests for userRequestSummaries Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: comments Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: formatting --------- Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> Co-authored-by: 0xfarhaan <59924029+0xfarhaan@users.noreply.github.com>
* fix: keep track of withdraw requests per user * feat: keep only active redeem requests in userRequests array * fix: update failing tests and fix indexes * fix: bring back console import * chore: import style * chore: library format * feat: upgrade removeRequest test for more complex scenario * fix: remove userRequestSummaries and bring back userRequests function * chore: rename mapping * fix: remove isPresent mapping * fix: renamings * feat: unit tests for library * fix: remove empty lines at the end of the files * Revert "fix: remove empty lines at the end of the files" This reverts commit 312e3207f9b6a17a8944d9540dc2273156bd9968. * chore: format * chore: formatting * fix: add extra require statements * feat: add invalid range test * fix: rename removeByValue to remove * fix: remove padination request getter * chore: Fix formatting, return value and test ordering * chore: Add TODO --------- Co-authored-by: 0xfarhaan <59924029+0xfarhaan@users.noreply.github.com>
…57) (#35) * feat: update removeShares to iterate over all user's requests * chore: format * chore: remove unnecessary return statement * fix: update push tests for SortedArray library * chore: formatting --------- Co-authored-by: 0xfarhaan <59924029+0xfarhaan@users.noreply.github.com>
* feat: added migrator * feat: added unit tests Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: unit tests and migrator Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: formatting Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: formatting Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: formatting Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: formatting Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: fixed storage layout bug Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: rename and formatting Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * chore: removed file Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * fix: fix test Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> --------- Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com>
* feat: changed wm interfaces to use uint256 Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: more casting Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * pr fixes Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> --------- Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com>
* feat: Add removeSharesById Functionality * fix: err messages + test cases * feat: Add additional require * chore: format * fix: test cases * feat: Update Interface and Tests * feat: Update Test with multiple LPs * chore: fix formatting * chore: fix stack too deep * chore: fix test name * chore: update impl * chore: address PR comments * chore: fix events
* feat: changed access control Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> * feat: added tests Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com> --------- Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com>
* feat: update event per OC request * feat: Internal audit fixes * feat: fix foundry TOML * fix: comment out fuzz tests * fix: comment out fuzz tests * fix: comment out fuzz tests * fix: Alignment
* Audit: Add comment clarifying usage for removeRequest on WM * chore: add comment
* fix: add bot support for removeRequests() * chore: Fix format --------- Co-authored-by: 0xfarhaan <59924029+0xfarhaan@users.noreply.github.com>
* feat: Swap Array to LinkedList * feat: update sorted linked list * fix: rename array to list * fix: remove get function from library * fix: cache list size * chore: format: * fix: remove unneccesary casting --------- Co-authored-by: Danilo <kitanovic.danilo@outlook.com>
* feat: Interal audit fixes * feat: ASCII Art
Signed-off-by: calmacfadden <cathalmf+calmacfadden@gmail.com>
* audit: Fix DOS Issue * chore: order event * chore: order functions * fix: PR comments * fix: PR comments * fix: code comment
Sherlock AI |
|
LCOV of commit
|
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.
No description provided.