HomePhabricator

(stable) Promote 2016 Week 15

Description

(stable) Promote 2016 Week 15

Details

Auditors
CodeMouse92
chad
Group Auditors
Cowboy Commits
Provenance
epriestleyAuthored on Apr 8 2016, 11:36 PM
epriestleyPushed on Apr 8 2016, 11:36 PM
Parents
rP57e606b39575: Update Settings/Config UI
rPfea2389066ed: (stable) Improve error and header behaviors for Mailgun received mail webhook
Branches
Unknown
Tags
Unknown
Build Status
Buildable 11611
Build 14518: Run Core Tests

Event Timeline

CodeMouse92 added a subscriber: CodeMouse92.

This is the last update to hit my server, and now both here and there...

  1. Calendar app is throwing a Runtime Error. (Just go to the app to see it.)
  2. One cannot create new Bug Reports. (Just attempt to create a new bug report to see that).

Everything was functional yesterday, but I update to the latest stable each morning, so as I mentioned, this is the last update to hit my server. I haven't fully tested everything, but I can update here (or on bug reports, once that's working again) if I find anything else.

I can't reproduce anything on this server on the task issue, just on calendar.

FWIW I can't reproduce either in a local environment. Are you seeing these issues locally or just this server?

Hi @chad, okay...that's creepy. I had the error on this server when I tried to submit the bug report a little bit ago. Now submitting an identical report, it's gone.

Meh.

As to the Calendar problem, that's both here AND on my local installation (running on the latest stable).

(My Calendar bug report: T10766: Calendar - Unhandled Exception)

concern i believe can be cleared now

epriestley requested verification of this commit.Jan 27 2017, 5:20 PM

Concern doesn't appear to describe a reproducible issue.

This commit now requires verification by auditors.Jan 27 2017, 5:20 PM
This commit now requires verification by auditors.Jan 27 2017, 5:22 PM

This is a bug, albeit a harmless one. The second state transaction should not be posted, since the overall audit state didn't actually change:

Screen Shot 2017-01-27 at 9.22.32 AM.png (163×397 px, 23 KB)

This commit now requires verification by auditors.Jan 27 2017, 5:23 PM

For what it's worth, I haven't seen the problem I described at all recently. It must have been resolved (or there was a ghost living in my server...which I'm starting to suspect.)

All concerns with this commit have now been addressed.Jan 27 2017, 7:11 PM

Merged Changes

This commit merges a very large number of changes. Only the first 50 are shown.