Page MenuHomePhabricator

Plans: Phriction modern "v3" APIs, Drafts
Open, NormalPublic

Description

See PHI840. This may unravel somewhat, but the immediate path forward looks like adding draft phases and a publish operation.


This probably mostly revolves around an EditEngine upgrade. Other EditEngine stuff:

  • See T4274. This may be trivial after EditEngine.
  • See T5811. The modern draft stuff has better behavior.
  • See T9102, maybe. I think just declining this is also reasonable.
  • See T3963. EditEngine forms can roughly approximate this.
  • See T10859. Bundling additional data in the preview would be nice.

See T5410. I'm likely to just decline this.

History might need to become a real object?

  • See T12193. Notes could be comments? See also T1894. But we can pull mentions out of them, at a minimum.

See T4815. This is a minor but reasonable UI improvement.


Done

See PHI366. An instance wants more reasonable API access to Phriction. Phriction has weird v2 APIs and should have modern v3 APIs.

See T11163. I'll likely decline this.

See T10969. This is just a bug.

See T5941. Requires ferret or ngrams but those seem reasonable.

See T12344. This is "wrong" but we can probably make this work (maybe?) since "%" isn't a valid slug character anyway.

The contentID property on Document should probably not be nullable. We can generate a Document PHID first, save the Content with the right Document PHID, then save the Document with the content ID. Deleted/moved/stub documents still have Content, I'm pretty sure -- oh, except we can't do, this because Content uses documentID, not documentPHID. So maybe kick the can down the road. (This was resolved by swapping everything to PHIDs.)

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
epriestley added a revision: Restricted Differential Revision.Feb 15 2018, 1:33 PM
epriestley added a revision: Restricted Differential Revision.Feb 15 2018, 1:50 PM
epriestley added a revision: Restricted Differential Revision.Feb 15 2018, 2:29 PM
epriestley added a revision: Restricted Differential Revision.Feb 15 2018, 4:31 PM
epriestley added a revision: Restricted Differential Revision.Feb 15 2018, 5:49 PM
epriestley added a revision: Restricted Differential Revision.Feb 15 2018, 8:56 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 1:20 AM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 1:39 AM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 1:44 AM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 1:55 AM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 2:13 AM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 2:15 AM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 2:19 AM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 2:23 AM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 1:18 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 2:44 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 3:50 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 5:26 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 5:42 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 5:55 PM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 7:01 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 7:02 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 7:05 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 7:08 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 7:37 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 7:40 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 7:45 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 8:14 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 8:32 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 8:33 PM
epriestley added a revision: Restricted Differential Revision.Feb 16 2018, 8:41 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 16 2018, 8:42 PM
epriestley added a revision: Restricted Differential Revision.Feb 22 2018, 6:55 PM
epriestley added a commit: Restricted Diffusion Commit.Feb 22 2018, 8:49 PM

The corresponding support ticket was likely addressed by these changes. The remaining stuff here is still planned, but probably not moving forward any time soon.

epriestley added a revision: Restricted Differential Revision.Aug 28 2018, 9:00 PM
epriestley added a revision: Restricted Differential Revision.Aug 28 2018, 9:51 PM
epriestley added a commit: Restricted Diffusion Commit.Aug 28 2018, 9:53 PM
epriestley added a commit: Restricted Diffusion Commit.Aug 28 2018, 9:58 PM
epriestley added a revision: Restricted Differential Revision.Aug 28 2018, 10:14 PM
epriestley added a commit: Restricted Diffusion Commit.Aug 29 2018, 2:32 PM
epriestley added a revision: Restricted Differential Revision.Aug 29 2018, 3:18 PM
epriestley added a revision: Restricted Differential Revision.Aug 29 2018, 3:29 PM
epriestley added a revision: Restricted Differential Revision.Aug 29 2018, 3:47 PM

contentID on Document should probably not be nullable. We can generate a Document PHID first, save the Content with the right Document PHID, then save the Document with the content ID. Deleted/moved/stub documents still have Content, I'm pretty sure -- oh, except we can't do, this because Content uses documentID, not documentPHID. So maybe kick the can down the road.

D19619 fixes this, but now we have a different problem: if you create a new document and want the new document to be a draft, we have no way to represent that. I'm just going to not support that for now and figure it out later if there's interest in it.

epriestley added a revision: Restricted Differential Revision.Aug 29 2018, 4:34 PM
epriestley added a commit: Restricted Diffusion Commit.Aug 29 2018, 8:41 PM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.Aug 29 2018, 8:47 PM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a revision: Restricted Differential Revision.Aug 30 2018, 5:09 PM
epriestley added a commit: Restricted Diffusion Commit.Aug 30 2018, 5:12 PM
epriestley added a revision: Restricted Differential Revision.Aug 30 2018, 5:34 PM
epriestley added a commit: Restricted Diffusion Commit.Aug 30 2018, 5:37 PM
epriestley added a revision: Restricted Differential Revision.Aug 31 2018, 5:08 PM
epriestley added a commit: Restricted Diffusion Commit.Aug 31 2018, 5:30 PM
epriestley renamed this task from Plans: Phriction modern "v3" APIs to Plans: Phriction modern "v3" APIs, Drafts.Sep 5 2018, 2:35 PM
epriestley updated the task description. (Show Details)
epriestley added a revision: Restricted Differential Revision.Sep 11 2018, 3:15 PM
epriestley added a revision: Restricted Differential Revision.Sep 11 2018, 3:40 PM
epriestley added a revision: Restricted Differential Revision.Sep 11 2018, 4:39 PM
epriestley added a revision: Restricted Differential Revision.Sep 11 2018, 5:07 PM
epriestley added a revision: Restricted Differential Revision.Sep 11 2018, 5:28 PM
epriestley added a commit: Restricted Diffusion Commit.Sep 11 2018, 8:32 PM
epriestley added a revision: Restricted Differential Revision.Sep 11 2018, 8:39 PM
epriestley added a commit: Restricted Diffusion Commit.Sep 12 2018, 8:21 PM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.Sep 12 2018, 8:30 PM
epriestley added a commit: Restricted Diffusion Commit.Sep 12 2018, 8:45 PM
epriestley added a commit: Restricted Diffusion Commit.Sep 12 2018, 8:56 PM
epriestley added a revision: Restricted Differential Revision.Sep 13 2018, 3:33 PM
epriestley added a commit: Restricted Diffusion Commit.Sep 13 2018, 9:12 PM