New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extended nav user widget #124

Merged
merged 2 commits into from Sep 17, 2017

Conversation

3 participants
@fr05t1k
Contributor

fr05t1k commented Jul 13, 2016

  • feature: adding links into widget
  • feature: disabling logout and profile links
@shakaran

This comment has been minimized.

Show comment
Hide comment
@shakaran

shakaran Sep 15, 2017

Collaborator

@chteuchteu this PR looks clean to me as nice as feature, but probably you want to check fine all the details before to merge. It has clean merge too. Thanks

Collaborator

shakaran commented Sep 15, 2017

@chteuchteu this PR looks clean to me as nice as feature, but probably you want to check fine all the details before to merge. It has clean merge too. Thanks

@chteuchteu

This comment has been minimized.

Show comment
Hide comment
@chteuchteu

chteuchteu Sep 15, 2017

Collaborator

@shakaran This needs a bit more digging and testing indeed :/

Collaborator

chteuchteu commented Sep 15, 2017

@shakaran This needs a bit more digging and testing indeed :/

@shakaran shakaran merged commit 8a2e6c4 into avanzu:feature/2.0 Sep 17, 2017

@shakaran

This comment has been minimized.

Show comment
Hide comment
@shakaran

shakaran Sep 17, 2017

Collaborator

@fr05t1k checked and merged to master. Thanks for your contribution!

Collaborator

shakaran commented Sep 17, 2017

@fr05t1k checked and merged to master. Thanks for your contribution!

@shakaran shakaran added this to the Version 2.0 Stable milestone Sep 17, 2017

@shakaran shakaran added 4 - Done and removed 3 - Review labels Sep 17, 2017

@shakaran shakaran self-assigned this Sep 17, 2017

@shakaran shakaran self-requested a review Sep 17, 2017

@fr05t1k

This comment has been minimized.

Show comment
Hide comment
@fr05t1k

fr05t1k Sep 17, 2017

Contributor

It took more than a year 😄

Contributor

fr05t1k commented Sep 17, 2017

It took more than a year 😄

@shakaran

This comment has been minimized.

Show comment
Hide comment
@shakaran

shakaran Sep 17, 2017

Collaborator

@fr05t1k yes, sorry for that, the original author doesn't have enough time. I take the seat as collaborator. I will try to speed up the things here ;)

Collaborator

shakaran commented Sep 17, 2017

@fr05t1k yes, sorry for that, the original author doesn't have enough time. I take the seat as collaborator. I will try to speed up the things here ;)

@shakaran shakaran removed their request for review Sep 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment