Page MenuHomePhabricator
Feed All Stories

Sep 21 2016

• jmarcelo.sf added a comment to Q493: How Phabricator checks if deamons are being run?.

It's a persistent problem. I've already tried to restart the deamons, the OS itself, but the problem persists. Phabricator was installed in a CentOS 7.2 system following the steps in this link: https://www.vultr.com/docs/how-to-install-and-configure-phabricator-on-centos-7

Sep 21 2016, 4:30 PM · Daemons
• jmarcelo.sf updated the question details for Q493: How Phabricator checks if deamons are being run?.
Sep 21 2016, 4:25 PM · Daemons
chad added a comment to T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.

How can we reproduce this locally? Do you have a specific version of Outlook / OS you are using?

Sep 21 2016, 4:08 PM · Bug Report
epriestley closed T11668: Herald editing view show unknown error as Invalid.

This isn't a bug in Phabricator, then. Please review Contributing Bug Reports and Support Resources carefully for details on what we provide support with.

Sep 21 2016, 3:40 PM · Bug Report (Needs Information)
epriestley added a comment to T4190: Rebuild remarkup image proxying.

On the cache:

Sep 21 2016, 3:38 PM · Contributor Onboarding, Remarkup
bzerr created T11678: Inline images from emails sent from Outlook are not correctly attached to Maniphest Tasks.
Sep 21 2016, 3:36 PM · Bug Report
rabahmeradi added a comment to Q493: How Phabricator checks if deamons are being run?.

I have run to the same problem. I have restart the daemon multiple times to fix it.

Sep 21 2016, 3:30 PM · Daemons
rabahmeradi added a comment to Q493: How Phabricator checks if deamons are being run?.

I have run to the same problem. I have restart the daemon multiple times to fix it.

Sep 21 2016, 3:30 PM · Daemons
20after4 added a comment to T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form.

We use a text field for a 'reference number' which refers to tasks imported from bugzilla reference=bz123, or RT with reference=rt456

Sep 21 2016, 3:14 PM · Custom Fields, Maniphest, Bug Report
epriestley created T11677: Queue tasks from importing repositories at a lower priority level.
Sep 21 2016, 2:54 PM · Diffusion, Daemons
chad added a comment to Q493: How Phabricator checks if deamons are being run?.

Is this a persistent problem, or one only around for a minute after you change a config setting or update?

Sep 21 2016, 2:43 PM · Daemons
epriestley added a comment to T11676: For task with many dependencies, cannot scroll horizontally in Task Graph for certain widths (overflow is clipped).

I think this is the CSS that tries to cut off the titles interacting weirdly with other table CSS.

Sep 21 2016, 2:37 PM · Maniphest, Bug Report
• jmarcelo.sf updated Q493: How Phabricator checks if deamons are being run? from to How Phabricator checks if deamons are being run?.
Sep 21 2016, 2:30 PM · Daemons
epriestley added a comment to T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form.

I believed this was semi-intentional, although the downstream report suggests it worked at one point, so maybe I just made a mistake.

Sep 21 2016, 2:23 PM · Custom Fields, Maniphest, Bug Report
jcox added a revision to T4190: Rebuild remarkup image proxying: Unknown Object (Differential Revision).
Sep 21 2016, 2:02 PM · Contributor Onboarding, Remarkup
epriestley added a comment to T4190: Rebuild remarkup image proxying.

In this case, use the AphrontWriteGuard::beginScopedUnguardedWrites() method to surround the block -- you can grep for usage, but it will look like this:

Sep 21 2016, 1:15 PM · Contributor Onboarding, Remarkup
jcox added a comment to T4190: Rebuild remarkup image proxying.

I've got the endpoint and controller set up, but I get the following error when I call PhabricatorFile::newFromFileDownload:

Sep 21 2016, 1:12 PM · Contributor Onboarding, Remarkup
chad closed T11674: Phabricator mobile UI does not show dashboard that is installed as Invalid.

Click "home". to see your installed dashboard. This is the expected UI.

Sep 21 2016, 12:19 PM · Bug Report
aklapper created T11676: For task with many dependencies, cannot scroll horizontally in Task Graph for certain widths (overflow is clipped).
Sep 21 2016, 10:32 AM · Maniphest, Bug Report
aklapper created T11675: "Text" type custom field in Maniphest not displayed in Maniphest Advanced Search form.
Sep 21 2016, 10:08 AM · Custom Fields, Maniphest, Bug Report
• KalleKyllonen created T11674: Phabricator mobile UI does not show dashboard that is installed .
Sep 21 2016, 9:48 AM · Bug Report
• KalleKyllonen added a comment to T11668: Herald editing view show unknown error .

I tried with fields available in the upstream, so it can not be reproduced with vanilla up-to-date- copy.

Sep 21 2016, 7:07 AM · Bug Report (Needs Information)
• guoqiang5337 added a member for Almanac: • guoqiang5337.
Sep 21 2016, 3:05 AM

Sep 20 2016

alexmv added a revision to T11248: ArcanistLinter's addLintMessage incorrectly normalizes path on Windows: Unknown Object (Differential Revision).
Sep 20 2016, 10:05 PM · Windows, Bug Report
• greedhector28 added a watcher for Blessed Committers: • greedhector28.
Sep 20 2016, 9:57 PM
• greedhector28 added a watcher for Blessed Reviewers: • greedhector28.
Sep 20 2016, 9:55 PM
chad merged T11673: Timeline context menu doesn't disappear when taking action into T10400: Dropdown menus can overlap.
Sep 20 2016, 9:51 PM · PHUI, Bug Report
chad merged task T11673: Timeline context menu doesn't disappear when taking action into T10400: Dropdown menus can overlap.
Sep 20 2016, 9:51 PM · Bug Report
cspeckmim created T11673: Timeline context menu doesn't disappear when taking action.
Sep 20 2016, 6:27 PM · Bug Report
avivey updated the task description for T11668: Herald editing view show unknown error .
Sep 20 2016, 6:04 PM · Bug Report (Needs Information)
jcox awarded T9900: Figlet does not properly render apostrophes with some characters a Baby Tequila token.
Sep 20 2016, 5:28 PM · Contributor Onboarding, Remarkup
epriestley awarded T9900: Figlet does not properly render apostrophes with some characters a Cup of Joe token.
Sep 20 2016, 5:22 PM · Contributor Onboarding, Remarkup
jcox added a comment to T9900: Figlet does not properly render apostrophes with some characters.

Alright I've been able to narrow this down a little bit more. There's some error with how external/pear-figlet/Text/Figlet.php is smushing characters. Specifically lines 339-362 as well as the _rep($r) function. The equivalent function in the C code is here. I haven't been able to find an obvious discrepancy, but I suspect there's a subtle difference in how one of the regex functions is handling things or an error in one of the regexes themselves. This error also occurs with backticks and double-quotes:

Sep 20 2016, 5:08 PM · Contributor Onboarding, Remarkup
epriestley closed T11665: repo002.phacility.net is heavily loaded as Resolved.

All queues have emptied, and repo002 is back in a healthy range again:

Sep 20 2016, 4:39 PM · Ops, Phacility
epriestley edited the content of Phacility Cluster Devices.
Sep 20 2016, 4:36 PM
epriestley edited the content of Phacility Cluster Devices.
Sep 20 2016, 4:36 PM
sascha-egerer added a comment to T10725: Can't access binary files of differential revision.

I have that problem again. There is no relation between the file and the Diff. Don't know how that can happen...

Sep 20 2016, 4:06 PM · Restricted Project, Differential, Bug Report
epriestley added a revision to T11672: Evaluate persistent connections from HTTP contexts: Unknown Object (Differential Revision).
Sep 20 2016, 2:34 PM · Restricted Project, Performance, Clusters, Database
epriestley added a revision to T11672: Evaluate persistent connections from HTTP contexts: Unknown Object (Differential Revision).
Sep 20 2016, 2:29 PM · Restricted Project, Performance, Clusters, Database
epriestley created T11672: Evaluate persistent connections from HTTP contexts.
Sep 20 2016, 1:54 PM · Restricted Project, Performance, Clusters, Database
jcox closed T10673: Convert Phurl to use EditEngine as Resolved by committing Restricted Diffusion Commit.
Sep 20 2016, 1:39 PM · Contributor Onboarding, Restricted Project, Phurl
epriestley merged task T11671: Passing an incorrectly formatted revision ID to `arc patch` results in an exception into T8937: Unclear exception when patching non-existant revision.
Sep 20 2016, 1:33 PM · Arcanist, Bug Report
epriestley merged T11671: Passing an incorrectly formatted revision ID to `arc patch` results in an exception into T8937: Unclear exception when patching non-existant revision.
Sep 20 2016, 1:33 PM · Arcanist
cspeckmim updated the task description for T11671: Passing an incorrectly formatted revision ID to `arc patch` results in an exception.
Sep 20 2016, 1:26 PM · Arcanist, Bug Report
cspeckmim created T11671: Passing an incorrectly formatted revision ID to `arc patch` results in an exception.
Sep 20 2016, 1:26 PM · Arcanist, Bug Report
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

rP9329e6a12d9f improved the rate at which the Herald queue is draining by approximately 10x.

Sep 20 2016, 1:10 PM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

I've moved the largest instance from repo002 to a new empty shard, which has put both shards into fairly reasonable range. Queues still aren't draining as quickly as I'd like/expect.

Sep 20 2016, 1:02 PM · Ops, Phacility
epriestley added a comment to T11670: Automate more of the migration process for moving instances across shards in the Phacility cluster.

I moved the largest instance from shard 002 to shard 006. This took about 35 minutes, most of which was backup/transfer time. I hit a couple of minor issues, but things appear to have worked properly overall.

Sep 20 2016, 12:47 PM · Ops, Phacility
jsonnull added a comment to T9640: Make Phabricator compatible with PHP7.

PHP 7.1 RC2 is now available, and the patch for async_signals was merged into what is presently the 7.1 branch.

Sep 20 2016, 11:57 AM · Prioritized, Restricted Project, Infrastructure
epriestley added a comment to T11670: Automate more of the migration process for moving instances across shards in the Phacility cluster.

I migrated a test instance, which appears to have worked properly. I'm going to run though things again, then move a large instance off repo002.

Sep 20 2016, 11:52 AM · Ops, Phacility
chad awarded T11670: Automate more of the migration process for moving instances across shards in the Phacility cluster a 100 token.
Sep 20 2016, 11:50 AM · Ops, Phacility
epriestley added a comment to T11670: Automate more of the migration process for moving instances across shards in the Phacility cluster.
  • Suspend the instance: Admin → Instance → Manage Instance → Suspend Instance
  • Stop the daemons:
Sep 20 2016, 11:50 AM · Ops, Phacility
epriestley added a comment to T11670: Automate more of the migration process for moving instances across shards in the Phacility cluster.

I've provisioned and deployed a new shard, but not yet opened it for allocations. I'm going to launch and then migrate a test instance next.

Sep 20 2016, 11:06 AM · Ops, Phacility
epriestley created T11670: Automate more of the migration process for moving instances across shards in the Phacility cluster.
Sep 20 2016, 10:24 AM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

I let the queues drain for a bit overnight. One of the large queues is down from ~16K items to ~12K items in the last 8 hours, but I don't want to just let this sit for 24 hours to "resolve" it. I'm going to look at moving instances across shards instead, which is a capability we need to automate eventually anyway.

Sep 20 2016, 10:06 AM · Ops, Phacility
epriestley added a comment to T11668: Herald editing view show unknown error .

Is this reproducible using fields available in the upstream? How can I reproduce this issue against a clean, vanilla, up-to-date copy of Phabricator with no custom code?

Sep 20 2016, 9:50 AM · Bug Report (Needs Information)
• KalleKyllonen added a comment to T11668: Herald editing view show unknown error .

We have defined that one. It is like our teams with the company. When new bug is created for example maps team, herald automatically sets the right person/team for the bug.

Sep 20 2016, 9:46 AM · Bug Report (Needs Information)
epriestley added a comment to T11668: Herald editing view show unknown error .

Where is "Platform--MUST-->" coming from? That isn't a field we provide in the upstream.

Sep 20 2016, 9:39 AM · Bug Report (Needs Information)
• KalleKyllonen created T11669: Dashboard drag and drop a test in patience.
Sep 20 2016, 9:25 AM · Restricted Project, Dashboards, Bug Report
• KalleKyllonen added a comment to T11668: Herald editing view show unknown error .

phabricator 51f8ec4487be02e290b1182a7cd5b4069fe7d49f (Tue, Sep 20) arcanist 9e82ef979e8148c43b9b8439025d505b1219e213 (Thu, Aug 25) phutil fa6519f4cea1c059bd381c20971cac413cc4760e (Thu, Sep 15)

Sep 20 2016, 9:12 AM · Bug Report (Needs Information)
• KalleKyllonen created T11668: Herald editing view show unknown error .
Sep 20 2016, 9:10 AM · Bug Report (Needs Information)
• KalleKyllonen added a watcher for Bug Report: • KalleKyllonen.
Sep 20 2016, 9:06 AM
stevex awarded T10038: Plan the mid-term pathway for unit-test/linter bindings a Party Time token.
Sep 20 2016, 8:13 AM · Lint, Arcanist
cinek created T11667: Default global syntax.filemap sets default filetypes to JavaScript for JSON configurations.
Sep 20 2016, 7:31 AM · Contributor Onboarding, Bug Report
lyahdav added a comment to T4171: Building reporting and data systems.

@ksmith it's a Python script that you would run on your local machine. If you have arcanist setup for development it should be pretty simple to get it working. Ideally it would just be a plugin, but I didn't look into that approach yet.

Sep 20 2016, 4:51 AM · Restricted Project, FreeBSD, Facts, Wikimedia
cspeckmim renamed T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured from Differential comment panel haunter does not allow scrolling to bottom of differential revision to Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured.
Sep 20 2016, 4:04 AM · Differential, Bug Report
cspeckmim added a comment to T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured.

Ok I have observed this now on this install, however only in the second haunt mode Z+Z:
https://secure.phabricator.com/D15067

Sep 20 2016, 4:01 AM · Differential, Bug Report
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

Looking at improving the raw performance of bin/repository update, one fixable issue is that we run git for-each-ref three times:

Sep 20 2016, 3:16 AM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

I also identified an instance with a bad HTTP hook which had managed to queue up 8,000 tasks in the queue. These already have scaling backoffs and likely didn't contribute a meaningful amount of load, but we could perhaps find a more graceful approach to handling bad webhook URIs during T11330.

Sep 20 2016, 3:05 AM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

I disabled the keys of the user running 30 clones/minute and notified the instance administrator. This appeared to create some headroom; I reverted the daemon pool size limits to the default and we're draining the queues at a somewhat more reasonable pace now.

Sep 20 2016, 3:03 AM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

@swisspol, that's what this refers to ("From T11666"):

Sep 20 2016, 2:22 AM · Ops, Phacility
swisspol added a comment to T11665: repo002.phacility.net is heavily loaded.

I don't know if this is related but since an hour or so, whenever I push I get an error even though the push appear to work (the commits show up in Diffusion):

$ git push
# Push received by "web.phacility.net", forwarding to cluster host.
# Waiting up to 120 second(s) for a cluster write lock...
# Acquired write lock immediately.
# Waiting up to 120 second(s) for a cluster read lock on "repo002.phacility.net"...
# Acquired read lock immediately.
# Device "repo002.phacility.net" is already a cluster leader and does not need to be synchronized.
# Ready to receive on cluster host "repo002.phacility.net".
Counting objects: 56, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (50/50), done.
Writing objects: 100% (56/56), 9.30 MiB | 1.84 MiB/s, done.
Total 56 (delta 32), reused 1 (delta 0)
# Released cluster write lock.
AphrontQueryException: #1364: Field 'messageCount' doesn't have a default value
To ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git
   015051e..67b218b  master -> master
error: failed to push some refs to 'ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git'
Sep 20 2016, 2:21 AM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

A contributing issue is that one instance appears to be making clone requests for approximately 30 repositories per minute to this shard. This is easy to rate limit, but probably not easy to rate limit in a generalized, not-a-cluster-hack way.

Sep 20 2016, 2:13 AM · Ops, Phacility
swisspol updated subscribers of T11666: Checkout is failing from Phacility.

@epriestley Did you see the AphrontQueryException: #1364: Field 'messageCount' doesn't have a default value problem too?

Sep 20 2016, 1:57 AM · Bug Report
swisspol added a comment to T11666: Checkout is failing from Phacility.

Hummm I don't know if it was a transient error or I just happened to magically fix it by force pushing to the repo, but git pulls now work again on my Mac and in CI.

Sep 20 2016, 1:55 AM · Bug Report
epriestley merged task T11666: Checkout is failing from Phacility into T11665: repo002.phacility.net is heavily loaded.
Sep 20 2016, 1:51 AM · Bug Report
epriestley merged T11666: Checkout is failing from Phacility into T11665: repo002.phacility.net is heavily loaded.
Sep 20 2016, 1:51 AM · Ops, Phacility
swisspol added a comment to T11666: Checkout is failing from Phacility.

Pushing appears to work (as in the commits appear in Diffusion) but errors are printed:

$ git push
# Push received by "web.phacility.net", forwarding to cluster host.
# Waiting up to 120 second(s) for a cluster write lock...
# Acquired write lock immediately.
# Waiting up to 120 second(s) for a cluster read lock on "repo002.phacility.net"...
# Acquired read lock immediately.
# Device "repo002.phacility.net" is already a cluster leader and does not need to be synchronized.
# Ready to receive on cluster host "repo002.phacility.net".
Counting objects: 11, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (11/11), done.
Writing objects: 100% (11/11), 1.90 KiB | 0 bytes/s, done.
Total 11 (delta 10), reused 0 (delta 0)
# Released cluster write lock.
PHP Fatal error:  Uncaught exception 'FilesystemException' with message 'Requested path '/core/log/hyper' is not writable.' in /core/lib/libphutil/src/filesystem/Filesystem.php:1107
Stack trace:
#0 /core/lib/libphutil/src/filesystem/Filesystem.php(650): Filesystem::assertWritable('/core/log/hyper')
#1 /core/lib/libphutil/src/filesystem/PhutilDeferredLog.php(183): Filesystem::createDirectory('/core//log/hype...', 493, true)
#2 /core/lib/libphutil/src/filesystem/PhutilDeferredLog.php(155): PhutilDeferredLog->write()
#3 [internal function]: PhutilDeferredLog->__destruct()
#4 {main}
  thrown in /core/lib/libphutil/src/filesystem/Filesystem.php on line 1107
To ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git
   d77b984..6eb6700  master -> master
error: failed to push some refs to 'ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git'
Sep 20 2016, 1:50 AM · Bug Report
swisspol created T11666: Checkout is failing from Phacility.
Sep 20 2016, 1:48 AM · Bug Report
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

One remaining issue with backoff/cooldown is that mirrors aren't included: a regularly-updated repository with broken mirrors still tries mirroring. This is somewhat more involved to fix, though (see T6131).

Sep 20 2016, 1:07 AM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

Those changes seem stable. Unclear how much they're really helping, but load is looking at least somewhat better. I'm going to dig into things a bit more and see if I can get a better sense of whether we're on the path to recovery or not.

Sep 20 2016, 1:06 AM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

D16574 retains update cooldowns (so we don't immediately update everything after a restart) and sets up for the following change.

Sep 20 2016, 12:32 AM · Ops, Phacility
epriestley added a revision to T11665: repo002.phacility.net is heavily loaded: Unknown Object (Differential Revision).
Sep 20 2016, 12:28 AM · Ops, Phacility

Sep 19 2016

epriestley closed T7865: After restarting, repository daemons all do a lot of work as Resolved by committing Restricted Diffusion Commit.
Sep 19 2016, 11:49 PM · Phacility
epriestley added a revision to T7865: After restarting, repository daemons all do a lot of work: Unknown Object (Differential Revision).
Sep 19 2016, 11:48 PM · Phacility
epriestley added a revision to T11665: repo002.phacility.net is heavily loaded: Unknown Object (Differential Revision).
Sep 19 2016, 11:48 PM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

Repository retries (failed mirrors, failed fetches) have no real backoff

Sep 19 2016, 11:03 PM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

I could alternatively throttle down the highly active instances (or all instances on this shard) but that's not too great.

Sep 19 2016, 10:51 PM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.
  • Maybe a good time to launch new shards? Last tier size increase was 10 months ago (late November, 2015) when I fixed the allocation algorithm to be not a huge wasteful pile of garbage that required us to constantly throw hardware at it.
    • Maybe the allocation algorithm should consider active users, not just active instances? I think the 002 shard has the most active users by a hefty margin, just by luck of the draw. But this will tend to be moot at scale.
    • Maybe start moving instances between shards? This seems hard to do without downtime today, and doing all the legwork to reduce downtime through clustering is foreboding.
Sep 19 2016, 10:38 PM · Ops, Phacility
epriestley created T11665: repo002.phacility.net is heavily loaded.
Sep 19 2016, 10:29 PM · Ops, Phacility
fushi created T11664: Add option to allow owner packages to be added as reviewers to diffs, when the diff author is in the owner package.
Sep 19 2016, 10:21 PM · Restricted Project, Feature Request
epriestley renamed T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained from Remarkup navigation component is not able to have text starting with equals sign to Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.
Sep 19 2016, 9:44 PM · Documentation, Remarkup
cspeckmim added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

Ohhh nice, thank you I missed that
=C124

Sep 19 2016, 9:32 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

And you can technically:

Sep 19 2016, 9:32 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

Oh, right. That works, but I think isn't documented:

Sep 19 2016, 9:31 PM · Documentation, Remarkup
cspeckmim added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

It's an off-case for sure. We have a ton of data identifiers which all end with an equals-sign and a couple letters/numbers, which we abbreviate when referring to the datasets. I was actually trying to render the equals sign as part of the nav sequence and not just C124.

Sep 19 2016, 9:30 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

We could conceivably special-case this since setting an empty attribute to a value isn't meaningful, I suppose.

Sep 19 2016, 9:25 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.
{nav name=C124}
Sep 19 2016, 9:24 PM · Documentation, Remarkup
cspeckmim created T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.
Sep 19 2016, 9:04 PM · Documentation, Remarkup
epriestley created T11662: Draft comments which reply to ghost inlines from lines which no longer exist can not be edited or deleted until the page is reloaded.
Sep 19 2016, 8:43 PM · Inline Comments, Customer Impact, Differential