Ref T13513. See that task for some discussion. This prepares to lift "loadUnsubmittedInlineComments(...)" into shared code.
Details
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
What is now programmatical way to "submit" unsubmitted inline comments? Some users are using it to post comments from bots, now if we use differential.createinline api method all bot comments are visible as unsubmitted and looks like there is no way to set it in stone...
I'm not aware of any current customer use cases for this behavior. I'm open to restoring it if customers are actually using it, but would like to make sure the API is fully supporting whatever the need is.
With newer inline comment functionality there is no ability to create submitted inline comments for bots or via API directly. differential.createinline call might simply always fully publish comments though it will create multiple transactions instead of one which was side effect of old attach_inline functionality. This could as well be differential.editoption to publish inline comments to make it more modern. We use inline comments it for smarter linting messages.
Inline commenting with edit suggestion (I suppose api is not ready for that) can be used to implement linters which offer autoformatting code...
With newer inline comment functionality there is no ability to create submitted inline comments for bots or via API directly.
Correct. I'm not currently aware of any customers who want support for this. If customers request this, I'm happy to revisit support.