-
Notifications
You must be signed in to change notification settings - Fork 26
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
Delete pending cap when removing a market #348
Merged
Merged
Changes from 4 commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
2985046
fix: delete pending cap when removing a market
Jean-Grimal ee787f9
Update test/forge/MarketTest.sol
Jean-Grimal f3aea6d
Update test/forge/MarketTest.sol
Jean-Grimal 03a0c10
Merge branch 'post-cantina' into fix/issue-491
Jean-Grimal d252fd9
fix: revert in submit cap when market is forced removed
Jean-Grimal cb23c75
fix: style
Jean-Grimal 7dff204
fix: apply suggestions
Jean-Grimal be004c2
Merge branch 'post-cantina' into fix/issue-491
Jean-Grimal ce1018e
Merge branch 'fix/issue-491' into feat/submit-cap-market-removal
Jean-Grimal ecdb902
feat: revert in set cap if removal has been submitted
Jean-Grimal 7520ae7
Merge branch 'fix/issue-491' of github.com:morpho-labs/morpho-blue-me…
Jean-Grimal 393798f
Merge branch 'fix/issue-491' into feat/submit-cap-market-removal
Jean-Grimal d18645a
Merge pull request #355 from morpho-org/feat/submit-cap-market-removal
MathisGD a68fdc8
fix: conflicts
Jean-Grimal 14a4d7d
fix: revert instead of delete pending cap
Jean-Grimal 810d0a2
fix: revert earlier
Jean-Grimal e95d7cc
refactor: remove removableAt check in _setCap
MathisGD 42494e3
refactor: reorder checks in submitMarketRemoval
MathisGD 7c4eaec
test: minor fix
MathisGD File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[optional] I realize now that these changes (both
config[id]
andpendingCap[id]
) may be difficult to follow offchain with events. This requires to do the difference of sets to know which markets were removed. Maybe emitting an event here would be nice for offchain purposesThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @julien-devatom