Page MenuHomePhabricator

Add profile images to Repositories
ClosedPublic

Authored by chad on Jun 12 2017, 3:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 18, 2:26 PM
Unknown Object (File)
Mon, Apr 8, 4:39 AM
Unknown Object (File)
Mar 20 2024, 3:00 PM
Unknown Object (File)
Mar 17 2024, 5:45 PM
Unknown Object (File)
Feb 18 2024, 9:33 AM
Unknown Object (File)
Feb 17 2024, 4:39 AM
Unknown Object (File)
Jan 29 2024, 12:17 PM
Unknown Object (File)
Jan 28 2024, 5:02 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.