Implemented for dialogs with yes/no/cancel, yes/no, ok/cancel actions. Any other dialogs with custom actions will return undefined exitActionType
(cherry picked from commit dd030c0a4a98c385d30c3710b3a063f500bb400e)
GitOrigin-RevId: f95022c182218f0172c7aa9a70aea38c2e44c420
#IJPL-175710 Fixed
This is caused by `... on Bot` not being allowed for `PullRequestReviewers` before GitHub 3.11.
The newly added tests can confirm this mistake.
(cherry picked from commit 87015863c0c8c0cd581f98eeefa5ae45adad34e8)
IJ-CR-154633
(cherry picked from commit 93d7bbd449b714c9a2f35c04aaf8e7168d73afcf)
IJ-CR-154633
GitOrigin-RevId: bc130a8de2e50e0faca7e1c1e89ec5063fc2b3cb
Every Actor subclass that exists as far as I could tell
now has a DTO representation and a fragment representation.
Sure, there's a lot of common classes, we can reuse actorInfo
in some cases, but I somewhat prefer having the specific fragment
to reference at the DTO class.
Just so that debugging these types of issues is easier later.
actor.graphql now directly represents the 5 possible instantiations of Actor.
(cherry picked from commit 6392ac4ce579ce48682d8cc288d2edfe5480da86)
IJ-CR-151664
(cherry picked from commit bd235370481269453094021d2e03ed91bb48903b)
IJ-CR-151664
GitOrigin-RevId: a6cc560b78552c1ca9003975c0b9fb03fd68b69d
Add actions:
- Generate a summary
- Like the generated summary
- Dislike the generated summary
Adds UI:
- A comment in timeline with a purple hover color
- buttons for above actions
(cherry picked from commit 6f734fb94f53a4efd8dd950b51d427a21f9ab117)
(cherry picked from commit 3e0ae846aa29182724b7a350a2567bff89070351)
IJ-CR-148445
GitOrigin-RevId: 51e1544114d334edfc5f001f9145e6d4a0c7e0ae
..and pass it to the existing review VM instead
of some specific VM exposing a specific path.
(cherry picked from commit c3fb55112fe4ff781d076d7d73ee1761fd97da56)
(cherry picked from commit 69ca8b42736fb88ffde195fea2e2e81f8f0e2fc0)
IJ-CR-148445
GitOrigin-RevId: 58b64b10ddcfa8a3339dba3164533940f54ca21d
This at least makes it possible to reload the review
and fixes the review loading immediately, even before user input.
Also adds a small warning in case of an exception.
No reload action included yet.
(cherry picked from commit 5a3cefacc08921b03dff2871e16eb88ba48e886b)
(cherry picked from commit c6b52c8d40a7f366fd71fd613b0bfc42575b193e)
IJ-CR-148445
GitOrigin-RevId: 14ee472fd7f0476df2c20b8d2dea10383fc7383b
It's basically unused and deprecated since CoroutineName is no longer used.
We practically always make a child scope with the class name.
Better to just use the class name in this case.
(cherry picked from commit 4827600b509f0ca804f5be1d05b35b11f8df470f)
(cherry picked from commit eb889a9e6dd0733a86664125feba158234b5267a)
IJ-CR-148445
GitOrigin-RevId: 5cf31226299b6ddad30f919b4ddeb36c8ae947aa
Toolbar was deterministically updated before adding it to the component tree.
`launch` is more likely to delay that update, though I guess it may not be 100%.
GitOrigin-RevId: 912684ffff5ade602b3b0448bdbd25a304426b23
`BrowseFolderRunnable` and `*WithBrowseButton` family: getting rid of separate "title" and "description" parameters in favor of the chooser descriptor object, to avoid cloning the latter down the road.
GitOrigin-RevId: 33ec5968a1db953c60848974135055c288accf85
The list of reviewers was simply emitted, but this
caused a dumb list updating algorithm to replace some entries in the list
without checking equality.
Reason is that it wants to put selected items at the front of the list.
Now, we emit the full list with those selected items at the front,
which prevents erroneous behavior.
GitOrigin-RevId: da238eec1eebd9428cabd3692a7349ea93324047
All diff requests created via `ChangeDiffRequestProducer#createRequest` will contain `DiffUserDataKeysEx.VCS_DIFF_RIGHT_CONTENT_TITLE` allowing to show short revision and file path relative to a project root
The same applies to diffs created via `MergedChangeDiffRequestProvider`
GitOrigin-RevId: 60ecb120e8751a6a211d2917838d6403a6003dfd