-
Notifications
You must be signed in to change notification settings - Fork 214
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
Imodel02 merge master #753
Merged
Merged
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
* Change iModelHub api release tags * extract-api fixes * extract-api changes * Change file * PR fixes Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
An invalid, or undefined, icon passed to the BrowserWindow causes a crash on MacOS with Electron 11, related Electron issue, electron/electron#27303.
… synchronizing changes from a branch back to master (#671) * Add IModelTransformOptions.isReverseSynchronization to better support synchronizing changes from a branch back to master.
* change to default behavior of IModelDb.close() and IModelDb.saveChanges() Co-authored-by: Affan Khan <khanaffan@users.noreply.github.com>
Co-authored-by: imodeljs-admin <38288322+imodeljs-admin@users.noreply.github.com>
* Initalize mobile BackendIpc before IModelHost.startup needs it * change * Do not rely on transport in constructor -- unnecessary entanglement with RPC world * api * missing event in handler call * test fix and uv event loop fix * lint Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* Improved AccuDraw input field, formatting using KOQs, updating value on mouse motion * Unit tests. Added FrameworkAccuDrawUiAdmin for hasInputFocus. * extract-api * Support for grabInputFocus * Minor fixes before merging in Raman's briefcase & Electron fixes * Fixed selecting values to UI * Unit test * Fixed value from UI on delay * Better AccuDraw input field focus & selection handling * rush change * Vertical/Horizontal layouts, multiple AccuDraw dialogs open, more rotate shortcuts, UiFramework.EscapeToHome setting * rush change * Changed ToolAdmin.onKeyTransition so onCtrlKeyPressed and processShortcutKey can be overridden * extract-api, unit tests, removed onKeyTransition from tools * Cleaned up isContextMenuOpen * Removed unused value handling from AccuDrawFieldContainer * Initializing value in AccuDrawInputField * Fixed KeyShortcutMenu keydown/keyup issue. Rearranged AccuDraw shortcuts - Rotation commands now under 'A' key. * extract-api * Fixed AccuDrawInputField unit test requestNextAnimation problem Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* Changed the relationship name used for preceding Checkpointv2 query * API break: changed the CheckpointV2Query.precedingCheckpoint method name to CheckpointV2Query.precedingCheckpointV2
* changes to allow target=_blank if secure relationship is present * add tests * changes * fix linter errors * fix coverage Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
#689) If `scrollToNode` is called within `useEffect` that is run immediately after `TreeRenderer` component is mounted, an assertion error occurs. This is due to contents of `TreeRenderer` being rendered in two passes: first it determines its container width and height, and then it mounts `VariableSizeList` on the second pass. This change introduces a new internal component called `VirtualizedList` which wraps `VariableSizeList` and adds the expected behaviour regarding `scrollToNode`. I don't think this is the ideal solution, the better one is to move `AutoSizer` component outside and let `TreeRenderer` consumers to specify tree width and height, but that would be a breaking change.
* Correctly handle capitalized SyncUiEventsIds. * Rush change.
* try increasing heap size * lower ui-components coverage threshold by .1%
…ackend and appNodeModules (#672) * Search for imodeljs-native in imodeljs-backend and not just appNodeModules * rush change Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* Update remaining syncEventId checks to be case insensitive.
* fix markdown lint errors * Apply markdownlint automated formatting * Consistently use `-` for lists instead of `*`.
* Disable importSorter.removeUnusedImports * move disable comments to above line by default * disable import sorter action on save * re-enable removeUnusedImports Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
allow briefcaseId to be provided to bridge runner Co-authored-by: admccarthy1 <admccarthy1@users.noreply.github.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Paul Connelly <22944042+pmconne@users.noreply.github.com>
* Conditionally disable/hide keyboard shortcuts * Rush change
* Presentation: Enable an option for presentation backend to use memory mapped i/o for worker SQLite connections (#690) * presentation/default units (#657) * Add default-units functionality and testing * change and API * PR Fixes * PR fixes * renaming * PR fix * style fix * PR fixes * NextVersion and minor fixes * PR fixes * minor fix * PR fixes * minor fix * Presentation: Enforce result paging for hierarchy compare (#678) * Enforce hierarchy compare result paging * extract-api * rush change * Restore broken public api * rush extract-api * Rename to 'continuationToken' * Add test for hierarchy compare paging * Update RPC version * Adjustments to latest ECDb changes for moving properties (#730) * rework IModelApp and IModelHost initialization (#687) * add ipc response handler * use responseHandler for NativeApp * extract-api * split out TxnManager * extract-api, again * remove EventSink and EventSource * wip * wip * get tests working * ElectronHost and ElectronApp * clean up startup code * clean up documenation * work on mobile-manager * wip * add mobile-manager * IOSApp and AndroidApp * directory case change * wip * wip * lint cleanup * use Ipc for StandaloneConnection * clean up documentation * rename callBackend to callXxxHost * clear subcategories in beforeClose rather than in each subclass * deprecate RemoteBriefcaseConnection, added RemoteIModelConnection * extract-api * rename RemoteIModelConnection to CheckpointConnection * more documentation * always import IModelStatus from bentleyjs-core in backend * fix presentation-backend test failures * Get presentation update-related tests working again * Update docs/learning/backend/IModelHost.md Co-authored-by: Grigas <35135765+grigasp@users.noreply.github.com> * Update core/common/tsconfig.json Co-authored-by: Grigas <35135765+grigasp@users.noreply.github.com> * add ProcessDetector * extract-api * make StandaloneConnection subclass of BriefcaseConnection * use IModelDb rather than BriefcaseDb for IModelWriteRpc * Working mockmobile tests * restore mobileeventloop * restored previous mobile ipc fix * eliminate StandaloneConnection class * fix openStandalone test * remove naked require('electron') in favor of ElectronHost.electron * fix for restarting IModelApp in InteractiveEditingSession.test * fix interactiveEditingSession test * fix IModelVersion interface * fix lint errors * steps toward getting mobile app working...still an issue in oidc * typo...authInit being called now. just need a MobileHost.device now, I think * move websocket ipc to mobilehost...still need to port device over * IOSDevice back in action. App launches...but no tiles??? * rework IOSHost.startup * typo * documentation for TxnManager * fix lint errors * rename StandaloneConnection test to OpenStandalone * ios/mobile refactoring * api * add test for classNameToId and classIdToName * NextVersion.md * typo Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Co-authored-by: Grigas <35135765+grigasp@users.noreply.github.com> Co-authored-by: swbsi <69857376+swbsi@users.noreply.github.com> * lint * extract-api * merge * lint errors * documenation descriptions * doc errors * documentation cleanup * revert ecschema-metadata changes Co-authored-by: Grigas <35135765+grigasp@users.noreply.github.com> Co-authored-by: aurislt7 <30312645+aurislt7@users.noreply.github.com> Co-authored-by: Saulius Skliutas <24278440+saskliutas@users.noreply.github.com> Co-authored-by: Robert Schili <rschili@users.noreply.github.com> Co-authored-by: kabentley <33296803+kabentley@users.noreply.github.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Co-authored-by: swbsi <69857376+swbsi@users.noreply.github.com> Co-authored-by: Caleb Shafer <31107829+calebmshafer@users.noreply.github.com>
kabentley
requested review from
abeesh,
bbastings,
bsteinbk,
c-macdonald,
calebmshafer,
ColinKerr,
diegoalexdiaz,
EarlinLutz,
khanaffan,
mdastous-bentley,
mgooding,
paulius-valiunas,
pmconne,
ramanujam-raman-zz,
RBBentley,
scsewall,
swbsi,
swwilson-bsi,
wgoehrig,
williamkbentley and
a team
as code owners
February 11, 2021 17:20
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.