Page MenuHomePhabricator

Use canonicalizer to display JSON
Closed, ResolvedPublic

Description

The front end displays JSON to the end-user in JSON mode and other places.

This JSON should be in the canonical format to make it a bit easier to read.

Related Objects

StatusSubtypeAssignedTask
OpenNone
OpenNone
StalledNone
Resolvedcmassaro
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
DuplicateNone
OpenNone
Resolvedcmassaro
OpenNone
ResolvedNone
ResolvedNone
Resolvedcmassaro
Resolvedcmassaro
ResolvedNone
Resolvedgengh
Resolvedgengh
ResolvedSimoneThisDot
ResolvedJdforrester-WMF
ResolvedDVrandecic
ResolvedNone
ResolvedNone
Resolvedcmassaro
ResolvedDVrandecic
ResolvedLindsaykwardell
Resolvedarthurlorenzi
ResolvedNone
InvalidNone
Resolvedcmassaro
Resolvedcmassaro
Resolvedcmassaro
Resolvedcmassaro
Resolvedcmassaro
ResolvedJdforrester-WMF
Resolvedcmassaro
Resolvedcmassaro
Resolvedcmassaro
ResolvedNone
ResolvedSimoneThisDot
ResolvedSimoneThisDot
ResolvedSimoneThisDot
ResolvedJdforrester-WMF
Resolvedcmassaro
Resolvedgengh
Resolvedcmassaro
ResolvedJdforrester-WMF
Resolvedcmassaro
Resolvedcmassaro
ResolvedNone
ResolvedDVrandecic
Resolvedcmassaro
Resolvedarthurlorenzi
ResolvedAdesojiThisDot
ResolvedNone
DeclinedLindsaykwardell
DeclinedNone
ResolvedLindsaykwardell
Resolvedcmassaro
Resolvedcmassaro

Event Timeline

DVrandecic renamed this task from Use the canonicalizer to display the JSON to Use canonicalizer to display JSON.May 13 2021, 8:09 PM
DVrandecic updated the task description. (Show Details)

This task is mostly complete, as I have a canonicalize function in the UI that performs this task. It is not reliant on the function-schemata at this time. I'm unsure whether the two canonicalize functions can be combined, as they handle different use cases (the canonicalizer in the frontend also handles undefined keys, for example, and has no additional validation/error reporting).

Based on our meeting yesterday, and Cory's work on implementing the schemata tests in the UI, I think we can safely say that this task is complete once the tests are in place.