Differential D1952

Improve ApiResponse implementation for QueryResult documentation
ClosedPublic
Actions

Authored by dereckson on Oct 23 2018, 00:59.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Oct 24, 16:43

	Unknown Object (File)
	Fri, Oct 24, 14:22

	Unknown Object (File)
	Thu, Oct 23, 15:19

	Unknown Object (File)
	Thu, Oct 23, 06:57

	Unknown Object (File)
	Tue, Oct 21, 04:32

	Unknown Object (File)
	Tue, Oct 21, 04:22

	Unknown Object (File)
	Tue, Oct 21, 04:20

	Unknown Object (File)
	Tue, Oct 21, 04:20

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rLFa5a06613692f: Improve ApiResponse implementation for QueryResult documentation

Summary

To avoid a too magic method, we explain comprehensively the error codes
through two examples, the current one for SELECT, a new one for INSERT.

The II and III code comments cases were swapped.

Test Plan

no op

Diff Detail

Repository

rLF Limiting Factor

Lint

Lint Passed

Unit

No Test Coverage

Branch

doc (branched from master)

Build Status

Buildable 3042
Build 3290: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Oct 23 2018, 00:59

dereckson created this revision.

Harbormaster completed remote builds in B3042: Diff 4936.Oct 23 2018, 00:59

dereckson accepted this revision.Oct 23 2018, 01:00

This revision is now accepted and ready to land.Oct 23 2018, 01:00

Closed by commit rLFa5a06613692f: Improve ApiResponse implementation for QueryResult documentation (authored by dereckson). · Explain WhyOct 23 2018, 01:00

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

api/

replies.rs

34 lines

Diff 4936

View Options

Improve ApiResponse implementation for QueryResult documentationClosedPublicActions