-
Notifications
You must be signed in to change notification settings - Fork 8
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
GRIDEDIT-1310 Added handling of multiple domains with undo #356
Merged
BillSenior
merged 36 commits into
master
from
feature/GRIDEDIT-1310_undo_for_multiple_domains
Aug 27, 2024
Merged
GRIDEDIT-1310 Added handling of multiple domains with undo #356
BillSenior
merged 36 commits into
master
from
feature/GRIDEDIT-1310_undo_for_multiple_domains
Aug 27, 2024
Conversation
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
BillSenior
force-pushed
the
feature/GRIDEDIT-1310_undo_for_multiple_domains
branch
from
July 25, 2024 07:55
bdb2bff
to
49b8605
Compare
BillSenior
force-pushed
the
feature/GRIDEDIT-1310_undo_for_multiple_domains
branch
from
July 29, 2024 08:56
09b217f
to
7177365
Compare
BillSenior
force-pushed
the
feature/GRIDEDIT-1310_undo_for_multiple_domains
branch
from
August 1, 2024 16:16
9ca8cbc
to
3d6ffc1
Compare
BillSenior
force-pushed
the
feature/GRIDEDIT-1310_undo_for_multiple_domains
branch
from
August 6, 2024 12:36
2954780
to
7161cab
Compare
…ps://github.com/Deltares/MeshKernel into feature/GRIDEDIT-1310_undo_for_multiple_domains
…arison of CLGs after undo
Add copy assignment operator for CLG (SC warning) Updated comments in UndoTests api unit test
changed api function name to be consistent.
lucacarniato
approved these changes
Aug 21, 2024
libs/MeshKernel/include/MeshKernel/UndoActions/UndoActionStack.hpp
Outdated
Show resolved
Hide resolved
…edChainages_ShouldGenerateMesh1D unit test
lucacarniato
requested changes
Aug 23, 2024
lucacarniato
approved these changes
Aug 26, 2024
Remvoed the enum indicating the current state of the mesh Removed the get null value function and all uses Updated unit tests to take changed into account.
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.