Page MenuHomePhabricator

Add profile images to Repositories
ClosedPublic

Authored by chad on Jun 12 2017, 3:31 AM.
Tags
None
Referenced Files
F14397119: D18116.diff
Sun, Dec 22, 9:32 AM
Unknown Object (File)
Wed, Dec 18, 8:36 AM
Unknown Object (File)
Mon, Dec 16, 6:19 PM
Unknown Object (File)
Fri, Dec 13, 1:59 PM
Unknown Object (File)
Fri, Dec 6, 3:43 AM
Unknown Object (File)
Fri, Dec 6, 2:54 AM
Unknown Object (File)
Tue, Dec 3, 10:07 PM
Unknown Object (File)
Sat, Nov 30, 11:56 AM
Subscribers

Details

Summary

Builds out some images to use to identify repositories. Fixes T12825.

Test Plan

Try setting custom, built in, and null images.

image.png (782×978 px, 75 KB)

image.png (552×924 px, 75 KB)

Diff Detail

Repository
rP Phabricator
Branch
diff-profile-image (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17476
Build 23434: Run Core Tests
Build 23433: arc lint + arc unit

Unit TestsFailed

TimeTest
227 msPhabricatorChangeParserTestCase::Unknown Unit Message ("")
EXCEPTION (Exception): Builtin "repository/repository1.png" does not exist! #0 /core/data/drydock/workingcopy-70/repo/phabricator/src/applications/files/storage/PhabricatorFile.php(1117): PhabricatorFilesOnDiskBuiltinFile->loadBuiltinFileData() #1 /core/data/drydock/workingcopy-70/repo/phabricator/src/applications/files/storage/PhabricatorFile.php(1167): PhabricatorFile::loadBuiltins(Object(PhabricatorUser), Array)
1 msAlmanacNamesTestCase::Unknown Unit Message ("")
30 assertions passed.
0 msAlmanacServiceTypeTestCase::Unknown Unit Message ("")
1 assertion passed.
1 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
1 assertion passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
3 assertions passed.
View Full Test Results (1 Failed · 355 Passed)

Event Timeline

  • fix failing harbormaster build

Are there license issues with these? They're from GitHub's icons, right?

This revision is now accepted and ready to land.Jun 12 2017, 2:14 PM

3 of the icons are, https://github.com/primer/octicons/blob/master/LICENSE

Do we need to add a license somewhere for dirivitive?

Maybe just put externals/octicons/ with the README and LICENSE?

This revision was automatically updated to reflect the committed changes.