Page MenuHomePhabricator

swisspol (Pierre-Olivier Latour)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
May 10 2014, 4:32 AM (266 w, 4 d)
Availability
Available

Recent Activity

Fri, May 24

trevorsummerssmith awarded T6004: When moving a wiki page with subpages prompt to also move subpages a Like token.
Fri, May 24, 1:54 PM · Phriction

Jul 25 2018

ekubischta awarded T5411: Allow pasting images when creating Pholio mocks a Orange Medal token.
Jul 25 2018, 3:25 PM · Pholio

Jul 5 2018

shuLhan awarded T5793: Add support for personal workboards a Like token.
Jul 5 2018, 9:51 AM · User Delight, Wikimedia, Workboards

Jun 22 2018

swisspol added a comment to D19505: Give "create" transactions a readable type in "transaction.search".

Will this affect commits i.e. will newly imported commits have a "create" transaction type?

Jun 22 2018, 5:45 PM
swisspol updated the summary of D19505: Give "create" transactions a readable type in "transaction.search".
Jun 22 2018, 5:43 PM

May 14 2018

tiguchi awarded T5793: Add support for personal workboards a Like token.
May 14 2018, 5:03 PM · User Delight, Wikimedia, Workboards

Feb 16 2018

mcorteel awarded T8824: Allow relative URLs in custom fields of type "link" a Like token.
Feb 16 2018, 1:56 PM · Custom Fields

Feb 11 2018

swisspol added a comment to T10746: Aborted build in Harbormaster still shows as building in Diffusion.

Is this retroactive to builds in this half state prior to this commit and
deploy?

Feb 11 2018, 12:17 AM · Restricted Project, Harbormaster, Bug Report

Feb 3 2018

swisspol added a comment to T10817: Allow more characters for built-in `filename` linter.

The above workaround is not reliable since it skips the files entirely vs checking the remaining characters.

Feb 3 2018, 7:58 PM · Lint, Feature Request

Aug 2 2017

brianespinosa awarded T5523: Make bulk moves easier on Workboards a Like token.
Aug 2 2017, 5:18 PM · Restricted Project, Workboards

Jul 6 2017

asherkin awarded T5523: Make bulk moves easier on Workboards a Like token.
Jul 6 2017, 9:01 AM · Restricted Project, Workboards

Jun 13 2017

swisspol added a comment to T10268: "Batch Edit" from workboards uses "?ids=X,Y,Z" which can exceed "Location" header or URL byte limits for large task sets.

Interestingly I worked around the issue by doing the batch edit from the Maniphest search UI which is passing the selected tasks an HTTP body instead of a URL query I guess?

Jun 13 2017, 10:02 PM · Customer Impact, Workboards, Maniphest

May 31 2017

alexmv awarded T5523: Make bulk moves easier on Workboards a Like token.
May 31 2017, 6:15 PM · Restricted Project, Workboards

May 23 2017

siepkes awarded T5793: Add support for personal workboards a Like token.
May 23 2017, 10:10 AM · User Delight, Wikimedia, Workboards

May 5 2017

beverly awarded T5353: Add visual hint to tasks that are blocked by others a Like token.
May 5 2017, 5:13 PM · Maniphest, Custom Fields

Apr 27 2017

azurasean awarded T5523: Make bulk moves easier on Workboards a Like token.
Apr 27 2017, 11:26 AM · Restricted Project, Workboards
n3v3rf411 awarded T5793: Add support for personal workboards a Like token.
Apr 27 2017, 6:54 AM · User Delight, Wikimedia, Workboards
arvinpan awarded T5793: Add support for personal workboards a Love token.
Apr 27 2017, 5:49 AM · User Delight, Wikimedia, Workboards

Apr 12 2017

swisspol added a comment to T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name.

Ah I missed that sentence above, this is why the bot user is disabled:

Mailing list and bot users are also considered "closed" because you usually want to hit normal users instead of them.

Apr 12 2017, 3:07 PM · Customer Impact, Typeahead, Bug Report
swisspol added a comment to T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name.

Another instance of the bug her:

Apr 12 2017, 3:02 PM · Customer Impact, Typeahead, Bug Report

Apr 11 2017

swisspol created T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name.
Apr 11 2017, 6:23 PM · Customer Impact, Typeahead, Bug Report

Mar 22 2017

swisspol added a comment to T11833: "parent"/"child" relationship for Revisions is confusing.

@epriestley If you just add copy in the selection dialog, it will help a lot IMO:

Mar 22 2017, 5:11 PM · Differential, Bug Report

Mar 21 2017

swisspol created T12426: Allow admins to edit the full names of users.
Mar 21 2017, 5:57 AM · Feature Request

Mar 20 2017

rraval awarded T5793: Add support for personal workboards a Like token.
Mar 20 2017, 6:42 PM · User Delight, Wikimedia, Workboards

Mar 19 2017

swisspol created T12419: Sorting doesn't work in Maniphest > Reports.
Mar 19 2017, 10:52 PM · Bug Report

Mar 18 2017

swisspol created T12415: Comment field action in batch action doesn't render correctly.
Mar 18 2017, 7:17 PM · Bug Report

Mar 4 2017

swisspol added a comment to T12350: Add support for "Plan Changes" as a condition for Differential Revision in Herald.

Got it thanks. And would a build be triggered by plan changes flag through the web form? Or only with another arc diff call?

Mar 4 2017, 3:17 PM · Feature Request
swisspol added a comment to T12350: Add support for "Plan Changes" as a condition for Differential Revision in Herald.

I understand why this may fall under T2543 but this other task appears to be of much bigger scope and take a very long time to be done, while this seems like a minor enhancement. Plan changes is just an attribute of the revision like summary, reviewers, etc... which you can use as conditions in herald, so why not just add it to herald.

Mar 4 2017, 3:03 PM · Feature Request
swisspol added a comment to T12350: Add support for "Plan Changes" as a condition for Differential Revision in Herald.

I don't know about that.

Mar 4 2017, 5:38 AM · Feature Request
swisspol created T12350: Add support for "Plan Changes" as a condition for Differential Revision in Herald.
Mar 4 2017, 4:45 AM · Feature Request

Mar 1 2017

swisspol added a comment to T12333: Arc diff is not creating a new revision when creating a new topic branch off a topic branch with already a revision associated.

Arcanist understands this is a new topic branch which no associated revision yet,

How?

Mar 1 2017, 12:07 AM · Bug Report

Feb 28 2017

swisspol closed T12332: Arc diff doesn't pick the right base commit and creates a new revision by mistake when working topic branch with merges as Invalid.

If you or your users sometimes use git push to mean "save changes", "make a backup", "share", etc., you can configure "Autoclose" in Diffusion. For example, you can configure only the "master" branch to Autoclose. If you configure Phabricator like this, it will never close revisions when changes are pushed to other branches.

Feb 28 2017, 8:22 PM · Bug Report
swisspol added a comment to T12333: Arc diff is not creating a new revision when creating a new topic branch off a topic branch with already a revision associated.

Expected flow:

  • have a topic branch and call arc diff
  • create a new topic off the first one then call arc diff again
  • Arcanist understands this is a new topic branch which no associated revision yet, so it does arc diff --create <first_topic_branch>
Feb 28 2017, 7:22 PM · Bug Report
swisspol added a comment to T12332: Arc diff doesn't pick the right base commit and creates a new revision by mistake when working topic branch with merges .

I understand the "why" but not the "why of the why" if that makes sense? What this means is that revisions might be closed in what appears to be unpredictable to users if they happen to push their topic branches to the repo AND have merge commits on them (that's the issue no?).

Feb 28 2017, 7:20 PM · Bug Report
swisspol added a comment to T12333: Arc diff is not creating a new revision when creating a new topic branch off a topic branch with already a revision associated.

Interestingly, using arc diff --create to force create a new revision generates one with a patch that still contains the diff of the original topic branch. The solution is to force arc diff to use the tip of the original topic branch as the base whenever calling arc diff while on this secondary topic branch.

Feb 28 2017, 6:39 PM · Bug Report
swisspol created T12333: Arc diff is not creating a new revision when creating a new topic branch off a topic branch with already a revision associated.
Feb 28 2017, 6:30 PM · Bug Report
swisspol added a comment to T12332: Arc diff doesn't pick the right base commit and creates a new revision by mistake when working topic branch with merges .

Clicking on "Explain Why" shows:

We didn't find a "Differential Revision" field in the commit message. This commit and the active diff of D127 had the same commit hash (70bfb4dd58c56a37296b00632113bd01d9facb83) so we linked this commit to D127.

Feb 28 2017, 6:11 PM · Bug Report
swisspol added a comment to T12332: Arc diff doesn't pick the right base commit and creates a new revision by mistake when working topic branch with merges .

Oh, it looks like pushing the topic branch when it was at the merge commit resulted in Diffusion auto-closing the Differential revision? This would explain the arc diff behavior reported above, but then the question becomes, why did it auto-close?

Feb 28 2017, 6:09 PM · Bug Report
swisspol created T12332: Arc diff doesn't pick the right base commit and creates a new revision by mistake when working topic branch with merges .
Feb 28 2017, 6:04 PM · Bug Report

Feb 27 2017

swisspol added a comment to T12325: Dynamically adjust the display precision of countdowns to increase dramatic tension.

I like that, sounds like a very good compromise.

Feb 27 2017, 8:35 PM · Customer Impact, Countdown, Feature Request
swisspol created T12325: Dynamically adjust the display precision of countdowns to increase dramatic tension.
Feb 27 2017, 8:01 PM · Customer Impact, Countdown, Feature Request

Feb 26 2017

swisspol added a comment to T10817: Allow more characters for built-in `filename` linter.

For the record, you can work around the problem as such:

{
  "linters": {
    "filename": {
      "type": "filename",
      "exclude": ["(\\+)", "(\\@)"]
    }
}
Feb 26 2017, 1:29 AM · Lint, Feature Request

Feb 11 2017

swisspol added a comment to T12234: Add a "Duplicates" tab to the "Related Objects" section on task page.

please no I am so young and have so much to live for

Feb 11 2017, 5:26 PM · Customer Impact, Feature Request
swisspol added a comment to T12234: Add a "Duplicates" tab to the "Related Objects" section on task page.

A possible workflow change might be to use "Edit Parent Tasks" instead of "Merge", to collect separate bugs under a single parent task but not actually close them.

Feb 11 2017, 1:55 AM · Customer Impact, Feature Request

Feb 9 2017

swisspol added a comment to T12234: Add a "Duplicates" tab to the "Related Objects" section on task page.

Why is it important to see all the duplicates at a glance?

Feb 9 2017, 8:23 PM · Customer Impact, Feature Request
swisspol updated the task description for T12234: Add a "Duplicates" tab to the "Related Objects" section on task page.
Feb 9 2017, 1:01 AM · Customer Impact, Feature Request
swisspol created T12234: Add a "Duplicates" tab to the "Related Objects" section on task page.
Feb 9 2017, 1:01 AM · Customer Impact, Feature Request

Feb 4 2017

lyahdav awarded T5523: Make bulk moves easier on Workboards a Like token.
Feb 4 2017, 1:48 AM · Restricted Project, Workboards

Jan 30 2017

swisspol closed T12170: Add plain comment to a Differential revision results in confusing error message as Invalid.

I just tried again (reloaded tab first) and didn't see the issue... Which is puzzling because I had reloaded the page twice and saw the issue twice before filing this bug.

Jan 30 2017, 10:23 PM · Bug Report
swisspol updated the task description for T12170: Add plain comment to a Differential revision results in confusing error message.
Jan 30 2017, 10:10 PM · Bug Report
swisspol created T12170: Add plain comment to a Differential revision results in confusing error message.
Jan 30 2017, 10:10 PM · Bug Report
souljedi awarded T5523: Make bulk moves easier on Workboards a Like token.
Jan 30 2017, 10:48 AM · Restricted Project, Workboards

Jan 25 2017

swisspol added a comment to T12154: PR branches are imported when mirroring a GitHub repo ignoring "Track Only" settings.

Yes, your email explains the real behavior under the hood. I filed this task from the perspective of what it looks to the user, not from the engineering implementation perspective. Feel free to change title / description of course.

Jan 25 2017, 5:53 PM · Bug Report
swisspol created T12154: PR branches are imported when mirroring a GitHub repo ignoring "Track Only" settings.
Jan 25 2017, 4:36 PM · Bug Report

Oct 19 2016

hartman awarded T5523: Make bulk moves easier on Workboards a Doubloon token.
Oct 19 2016, 10:00 AM · Restricted Project, Workboards

Oct 16 2016

swisspol added a comment to T11469: Checking out from Phacility sometimes fails.

Still failing as of today (rarely though):

Building in workspace /Users/admin/Jenkins/Home/workspace/Hyper-tvOS-AdHoc
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git # timeout=10
Fetching upstream changes from ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git
 > git --version # timeout=10
using GIT_SSH to set credentials Phabricator
 > git -c core.askpass=true fetch --tags --progress ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git +refs/heads/*:refs/remotes/origin/*
ERROR: Error fetching remote repo 'origin'
hudson.plugins.git.GitException: Failed to fetch from ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git
	at hudson.plugins.git.GitSCM.fetchFrom(GitSCM.java:763)
	at hudson.plugins.git.GitSCM.retrieveChanges(GitSCM.java:1012)
	at hudson.plugins.git.GitSCM.checkout(GitSCM.java:1043)
	at hudson.scm.SCM.checkout(SCM.java:485)
	at hudson.model.AbstractProject.checkout(AbstractProject.java:1276)
	at hudson.model.AbstractBuild$AbstractBuildExecution.defaultCheckout(AbstractBuild.java:607)
	at jenkins.scm.SCMCheckoutStrategy.checkout(SCMCheckoutStrategy.java:86)
	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:529)
	at hudson.model.Run.execute(Run.java:1738)
	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
	at hudson.model.ResourceController.execute(ResourceController.java:98)
	at hudson.model.Executor.run(Executor.java:410)
Caused by: hudson.plugins.git.GitException: Command "git -c core.askpass=true fetch --tags --progress ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git +refs/heads/*:refs/remotes/origin/*" returned status code 128:
stdout: 
stderr: ssh_exchange_identification: Connection closed by remote host
fatal: Could not read from remote repository.
Oct 16 2016, 9:48 PM · Phacility, Bug Report
swisspol updated the task description for T11469: Checking out from Phacility sometimes fails.
Oct 16 2016, 9:47 PM · Phacility, Bug Report

Oct 5 2016

amire80 awarded T5523: Make bulk moves easier on Workboards a Like token.
Oct 5 2016, 3:52 PM · Restricted Project, Workboards

Sep 27 2016

swisspol closed T11445: Commits with failed HarborMaster builds stuck in "Building" state in Diffusion as Invalid.

This has been solved AFAIK since I haven't seen this issue in a while.

Sep 27 2016, 5:12 PM · Bug Report

Sep 20 2016

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

Aug 29 2016

swisspol updated the task description for T11469: Checking out from Phacility sometimes fails.
Aug 29 2016, 9:57 AM · Phacility, Bug Report

Aug 28 2016

swisspol updated the task description for T11469: Checking out from Phacility sometimes fails.
Aug 28 2016, 8:04 AM · Phacility, Bug Report

Aug 15 2016

swisspol added a comment to T11445: Commits with failed HarborMaster builds stuck in "Building" state in Diffusion.

This is still happening (incidentally just observed because of the build failure due to T11469).

Aug 15 2016, 11:13 PM · Bug Report
swisspol added a comment to T11469: Checking out from Phacility sometimes fails.

Just observed a failure, see updated description.

Aug 15 2016, 10:44 PM · Phacility, Bug Report
swisspol updated the task description for T11469: Checking out from Phacility sometimes fails.
Aug 15 2016, 10:43 PM · Phacility, Bug Report

Aug 13 2016

swisspol added a comment to T11469: Checking out from Phacility sometimes fails.

Why do you mean by "this instance"?

Aug 13 2016, 2:27 PM · Phacility, Bug Report
swisspol updated the task description for T11469: Checking out from Phacility sometimes fails.
Aug 13 2016, 5:34 AM · Phacility, Bug Report

Aug 12 2016

swisspol created T11469: Checking out from Phacility sometimes fails.
Aug 12 2016, 8:58 PM · Phacility, Bug Report

Aug 8 2016

swisspol created T11445: Commits with failed HarborMaster builds stuck in "Building" state in Diffusion.
Aug 8 2016, 11:58 PM · Bug Report

Aug 4 2016

swisspol awarded M1466: Phriction v3 a Like token.
Aug 4 2016, 7:46 PM

Jul 18 2016

clemvangelis awarded T5793: Add support for personal workboards a Like token.
Jul 18 2016, 12:17 PM · User Delight, Wikimedia, Workboards

Jul 14 2016

timor awarded T5793: Add support for personal workboards a Like token.
Jul 14 2016, 11:40 AM · User Delight, Wikimedia, Workboards

Jul 7 2016

swisspol added a comment to T9412: Annotate all the reasons that a user is included on the recipient list for mail.

Until this is implemented, can someone tell me how to unsubscribe from Harbor Master build failure emails (see T11282)?

Jul 7 2016, 6:15 PM · llvm, Restricted Project, Mail

Jul 6 2016

swisspol created T11282: Clearly indicate in email footer how to unsubscribe from similar emails.
Jul 6 2016, 5:35 PM · Feature Request

Jul 1 2016

swisspol closed T11246: Commit stuck importing on Phacility as Resolved.

I was away for dinner and indeed, it was imported when I got back and everything seems good.!

Jul 1 2016, 1:54 AM · Bug Report
swisspol created T11246: Commit stuck importing on Phacility.
Jul 1 2016, 12:38 AM · Bug Report

Jun 3 2016

cmmata awarded T11069: Add support for Slack integration a Like token.
Jun 3 2016, 11:46 AM · Doorkeeper, Feature Request

Jun 1 2016

swisspol added a comment to T11069: Add support for Slack integration.

Is there a doc somewhere on how to configure this for Slack though? Can it post only Diffusion updates or does it have to post everything? If the latter, that's going to be way to noisy and won't work I'm afraid.

Jun 1 2016, 4:25 AM · Doorkeeper, Feature Request
swisspol added a comment to T11069: Add support for Slack integration.

Ah, I didn't know you had to edit it at https://admin.phacility.com/. I was looking at the instance config app.

Jun 1 2016, 4:23 AM · Doorkeeper, Feature Request
swisspol added a comment to T11069: Add support for Slack integration.

I don't think I can change this setting: "Configuration Locked". Can you guys do it for us?

Jun 1 2016, 4:00 AM · Doorkeeper, Feature Request
swisspol created T11069: Add support for Slack integration.
Jun 1 2016, 2:37 AM · Doorkeeper, Feature Request

May 30 2016

swisspol added a comment to T11051: "Explain Why" isn't properly explaining why revisions were closed.

Got it, thanks for taking the time to explain. When you say "metadata" for the commits, do you mean the commits are also in Diffusion (even if invisible)? If so, wouldn't this behavior affect Herald / Harbor Master?

May 30 2016, 4:34 PM · Restricted Project, Differential, Bug Report
swisspol added a comment to T11051: "Explain Why" isn't properly explaining why revisions were closed.

The expected behavior is that when we discover a new commit, we search for any open revision which has an active diff with a commit that has the same commit hash or tree hash as one of the discovered commits. In this case, it appears that this revision did.

May 30 2016, 4:03 PM · Restricted Project, Differential, Bug Report
swisspol added a comment to T11051: "Explain Why" isn't properly explaining why revisions were closed.

Feel free to look at this information and anything else in the Hyper instance you need to.

May 30 2016, 3:27 AM · Restricted Project, Differential, Bug Report

May 28 2016

swisspol created T11051: "Explain Why" isn't properly explaining why revisions were closed.
May 28 2016, 12:00 AM · Restricted Project, Differential, Bug Report

May 24 2016

swisspol added a comment to T11023: Branch contains condition in Herald rule not behaving as expected .

Deleting the branch first won't have any effect, since the commit still exists.

May 24 2016, 2:37 PM · Bug Report
swisspol added a comment to T11023: Branch contains condition in Herald rule not behaving as expected .

Sounds like a dup indeed. What's the work around?

  • Doing a rebase instead of FF-merge?
  • Deleting the topic branch in Diffusion first, then FF-merge and push?
May 24 2016, 2:29 PM · Bug Report
swisspol created T11023: Branch contains condition in Herald rule not behaving as expected .
May 24 2016, 6:14 AM · Bug Report

May 11 2016

swisspol created T10951: Search shows document types corresponding to disabled apps.
May 11 2016, 4:02 PM · Contributor Onboarding, Search, Bug Report

Apr 29 2016

swisspol added a comment to T10899: Print CSS does not fold/hide curtains for object pages like Phortune carts.

BTW is the print action the expected method to get an "invoice" for my records? Or can you download a formal invoice from somewhere?

Apr 29 2016, 3:08 PM · Design, Bug Report
swisspol created T10899: Print CSS does not fold/hide curtains for object pages like Phortune carts.
Apr 29 2016, 2:36 PM · Design, Bug Report

Apr 19 2016

swisspol added a comment to T10838: Differential revision page diff sometimes shows file diffs without visible differences.

OK I understand. Even if the current UX is not perfect, if implementing an improved version would result in a more fragile UX because the code is a lot more complex, then yes, the current situation is preferable.

Apr 19 2016, 3:24 PM · Bug Report
swisspol added a comment to T10838: Differential revision page diff sometimes shows file diffs without visible differences.

Ah I was wondering about this comment, but since it was explicitly hidden, I thought it wasn't the cause.

Apr 19 2016, 3:06 PM · Bug Report
swisspol created T10838: Differential revision page diff sometimes shows file diffs without visible differences.
Apr 19 2016, 2:47 PM · Bug Report

Apr 15 2016

swisspol added a comment to T10822: When vary-subjects is enabled, a task is merged, and subscribers are modifed the subject is misleading.

Thanks for looking into this @epriestley!

Apr 15 2016, 4:41 PM · Maniphest, Mail, Bug Report
swisspol added a comment to T10822: When vary-subjects is enabled, a task is merged, and subscribers are modifed the subject is misleading.

You can either solve this by not sending such emails when a task is merged (but they you don't notified of merged tasks which is bad), or by having a dedicated subject / body for merged task notifications, which then:

  1. solves this bug
  2. improves the UX since you now get a notification email that actually reflects the action that happened
Apr 15 2016, 3:59 PM · Maniphest, Mail, Bug Report