-
Notifications
You must be signed in to change notification settings - Fork 10
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
Thchang/2121 matching generated moment images toggle #2221
Thchang/2121 matching generated moment images toggle #2221
Conversation
@TienHao do you think it is possible to beautify the UI like the following? This would save some space. |
@kswang1029 Sure, just changed like this. |
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.
looks good! No regression is seen from e2e tests. Moment related tests are revised.
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.
Looks good. 👍
Description
Closes #2121.
Added a toggle in the moment generator to control the generated moment image(s) to be matched to the spatial reference image.
The toggle only shows up when the data source image is the spatial reference image.
The controlling property
momentToMatch
was added to AppStore.Checklist
For linked issues (if there are):
For the pull request:
e2e test passing/ corresponding fix addedno changelog update neededprotobuf updated to the latest dev commit/ no protobuf update neededBackendService
unchanged /__BackendService
changed and corresponding ICD test fix added