Skip to content
Snippets Groups Projects

Avatar

All threads resolved!
Merged Giovanni Caruso requested to merge avatar into master
All threads resolved!

Merge request reports

Pipeline #105678 passed

Pipeline passed for 74625d6a on avatar

Approval is optional

Merged by  Iurii  Maksymets Iurii Maksymets 5 years ago (Nov 15, 2019 2:01pm UTC)

Merge details

  • Changes merged into with 74625d6a.
  • Deleted the source branch.

Pipeline #105685 passed

Pipeline passed for 74625d6a on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • df793016 - Changed humanhash to be used on display

    Compare with previous version

  • added 1 commit

    • e3ee3d0f - Changed avatar generation to jdenticon

    Compare with previous version

  • Giovanni Caruso added 3 commits

    added 3 commits

    Compare with previous version

  • Looks great - but could you reduce the padding around the icons?

  • @gc4117 Could you finish up these small adjustments so we can merge your code?

  • added 1 commit

    • 468616d2 - Moved status indicator to the right of the user icon

    Compare with previous version

  • added 1 commit

    • 3bd78f5d - Changed humanhash to HumanHasher

    Compare with previous version

  • Giovanni Caruso added 6 commits

    added 6 commits

    Compare with previous version

  • Giovanni Caruso resolved all threads

    resolved all threads

  • added 1 commit

    • a8b1e97a - Fixed slider value not being updated on reloading

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Giovanni Caruso resolved all threads

    resolved all threads

  • Giovanni Caruso added 39 commits

    added 39 commits

    Compare with previous version

  • Giovanni Caruso added 3 commits

    added 3 commits

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • LGTU @ml5717 @alh1717 @mnr17

    Could you start a separate merge request after that to display the user's own name and icon somewhere as well?

  • Please register or sign in to reply
    Loading