HomePhabricator

(wip) Update question archive/visibility status via API
a02b2278715eUnpublished

Unpublished Commit · Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.
This commit no longer exists in the repository. It may have been part of a branch which was deleted.This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

(wip) Update question archive/visibility status via API

Before, we relied on a DeferredUpdate to make changes to question records stored
in the user's preference. TransactionProfiler is not happy about this as it's a
write on a GET.

This patch:

  • switches ApiQuestionStore to require a POST
  • only loads the questions (without updating) on GET
  • adds two POST requests at Homepage.js which will load and update questions

Bug: T222124
Change-Id: I21a485c75c6a7ff90985c3b3467907de5cde21a8

Details

Provenance
kostajhAuthored on May 1 2019, 8:31 PM
ChangeId
I21a485c75c6a7ff90985c3b3467907de5cde21a8

Commit No Longer Exists

This commit no longer exists in the repository.