Differential D1952

Improve ApiResponse implementation for QueryResult documentation
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F3993844: D1952.diff
	Fri, Jan 10, 13:44

	Unknown Object (File)
	Thu, Jan 9, 01:25

	Unknown Object (File)
	Wed, Jan 8, 13:30

	Unknown Object (File)
	Wed, Jan 8, 04:19

	Unknown Object (File)
	Tue, Dec 31, 15:42

	Unknown Object (File)
	Tue, Dec 17, 09:34

	Unknown Object (File)
	Fri, Dec 13, 04:06

	Unknown Object (File)
	Dec 8 2024, 13:51

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 Not Applicable

Unit

Tests Not Applicable

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 4937

View Options

Improve ApiResponse implementation for QueryResult documentationClosedPublicActions