Skip to content

Fix some details for pull request #136#1

Merged
michihenninger merged 2 commits intomichihenninger:prestige-pluginfrom
eduramiba:prestige-plugin
Nov 25, 2016
Merged

Fix some details for pull request #136#1
michihenninger merged 2 commits intomichihenninger:prestige-pluginfrom
eduramiba:prestige-plugin

Conversation

@eduramiba
Copy link
Copy Markdown

Thanks for the pull request gephi#136 :) You will need to accept a few changes before merging.

@michihenninger michihenninger merged commit b7189fb into michihenninger:prestige-plugin Nov 25, 2016
@michihenninger
Copy link
Copy Markdown
Owner

Hi @eduramiba . Thanks for reviewing my merge-request. Now I see how to use that processTicket ;-). Also thanks for all the effort you put into gephi plugin maintaining!
One last question before finally merging and accepting: Do you know why the report istn't show as a popup after the calculation? In all other statistics it works that way.

Thanks a lot and have a nice day!
Michi

@eduramiba
Copy link
Copy Markdown
Author

I don't know why the report is not showing. I spent some time trying to figure it out but I can't find the problem... 😭

@michihenninger
Copy link
Copy Markdown
Owner

michihenninger commented Nov 25, 2016

Hi @eduramiba . Thanks for the quick response. Don't worry, it's not that important. I'm quite happy to provide my first gephi plugin (and my first open source implementation). I am lecturer for Social Network Analysis and using gephi in my course. A great tool! Simple to use and powerful! Maybe more plugins are comming ;-)

@eduramiba
Copy link
Copy Markdown
Author

Awesome! :) I will merge your plugin soon.
If the problem is fixed later, the plugin can be easily updated.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants