Repository Commits #308

Merged
mmarif merged 8 commits from 13-repo-commits into master 2020-03-31 05:11:40 +00:00
Owner

Will close #13

Will close #13
mmarif added this to the 2.5.0 milestone 2020-03-25 14:52:15 +00:00
mmarif added the
Feature
WIP
labels 2020-03-25 14:52:15 +00:00
mmarif self-assigned this 2020-03-25 14:52:15 +00:00
mmarif added
LGTM-need
and removed
WIP
labels 2020-03-27 19:11:10 +00:00
Author
Owner

Screenshot of commits list

Screenshot of commits list
Author
Owner

@6543 Used your adivce to remove Commits tab. It's now under branch and have separate activity for listing them.

@6543 Used your adivce to remove Commits tab. It's now under branch and have separate activity for listing them.
6543 started working 2020-03-29 15:55:35 +00:00
Member

First checks: looks good but I need more time ...

First checks: looks good but I need more time ...
6543 stopped working 2020-03-29 16:07:11 +00:00
11min 36s
Member

By the way: a future impruvement: instead of the browser link for the "single commit diff view" we could add a single-commit-view ... but this is realy outside of scope of this pull-request (it gets to heavy)

By the way: a future impruvement: instead of the browser link for the "single commit diff view" we could add a single-commit-view ... but this is realy outside of scope of this pull-request (it gets to heavy)
Author
Owner

a future impruvement: instead of the browser link for the “single commit diff view” we could add a single-commit-view

Do you mean something like fileviewer or diff view?

> a future impruvement: instead of the browser link for the “single commit diff view” we could add a single-commit-view Do you mean something like fileviewer or diff view?
Author
Owner

looks good but I need more time

No worries, take your time.

> looks good but I need more time No worries, take your time.
6543 started working 2020-03-30 22:55:42 +00:00
6543 canceled time tracking 2020-03-30 23:11:55 +00:00
6543 started working 2020-03-30 23:11:58 +00:00
6543 approved these changes 2020-03-30 23:35:04 +00:00
Dismissed
6543 left a comment
Member

there was a lot of unrelated refactor in it too :/

there was a lot of unrelated refactor in it too :/
Member

there was a lot of unrelated refactor in it too :/ - but I hope I didn't missed something. It works on my side and I didn't found something while read throu the diff 👍

there was a lot of unrelated refactor in it too :/ - but I hope I didn't missed something. It works on my side and I didn't found something while read throu the diff :+1:
6543 added
LGTM-done
and removed
LGTM-need
labels 2020-03-30 23:35:55 +00:00
6543 stopped working 2020-03-30 23:36:04 +00:00
24min 7s
Author
Owner

Yes few unrelated changes but was working on it, so thought no harm to improve them. :)

Yes few unrelated changes but was working on it, so thought no harm to improve them. :)
mmarif closed this pull request 2020-03-31 05:11:40 +00:00
mmarif deleted branch 13-repo-commits 2020-03-31 05:11:51 +00:00
This repo is archived. You cannot comment on pull requests.
No reviewers
No Milestone
No Assignees
2 Participants
Total Time Spent: 35 minutes 43 seconds
6543
35 minutes 43 seconds
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: gitnex/GitNex#308
No description provided.