Skip to content
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

Improving readme.md in 1.3.0 #36

Merged
merged 8 commits into from
Oct 25, 2016
Merged

Improving readme.md in 1.3.0 #36

merged 8 commits into from
Oct 25, 2016

Conversation

Kurolox
Copy link
Contributor

@Kurolox Kurolox commented Oct 25, 2016

DO NOT MERGE YET!

It's not completed. Just leaving my progress here. I still have to fix the images and add more stuff.

Should fix #35 when finished.

@Kurolox Kurolox changed the title Swimlane/1.3.0 Improving readme.md in 1.3.0 Oct 25, 2016
@blackus3r blackus3r added this to the 1.3.0 milestone Oct 25, 2016
@blackus3r
Copy link
Owner

Just a notice: this should also land in the readme.txt file :) Thanks, for the pull Request.

You can use the same images from wordpress.

Here you find all images: https://plugins.svn.wordpress.org/post-worktime-logger/assets/

@blackus3r
Copy link
Owner

The readme.md is just for Github. The readme.txt is for wordpress, but at the moment, it's a bit empty on the page.

@Kurolox
Copy link
Contributor Author

Kurolox commented Oct 25, 2016

I've fixed the images. For some reason linking them directly from your link doesn't work, so I had to upload them to imgur. I'm not sure if you're okay with that, if you have any problem with it we could look for an alternative.

Also, may I ask what license are you using for this project? From the readme.txt I can't tell if you're using GPLv2 or GPLv3.

@blackus3r
Copy link
Owner

Thank you,

I will have a look later at it. (images)

Could you please adjust the readme.txt file too?

I see, I have a mismatch between the License tags. What do you think would be better? GPLv2 or v3?

@Kurolox
Copy link
Contributor Author

Kurolox commented Oct 25, 2016

Sure, I'll adjust the readme.txt, but I want to be fone with the readme.md first so I can make both of them similar. As I said, I still want to add a few things to the readme.md (like the license or a disclaimer).

About the license, honestly I don't have any idea. I'm really new to contributing, but I guess that GPLv3 would be better.

@blackus3r
Copy link
Owner

Okay, then we will take the GPLv3 licence. And we have to fix it in post-worktime-logger.php

@Kurolox
Copy link
Contributor Author

Kurolox commented Oct 25, 2016

Okay, everything should be done I believe. If you don't have any problem with the images being hosted in imgur, it should be ready to merge. Otherwise just tell me, we could do something like upload the screenshots to the repository and link them internally or something like that.

@blackus3r
Copy link
Owner

I have already thought about it several times, and I think it is a good solution to host the images in the repository.
So we have always the correct screenshots for the right version.
You can download the assets folder and add it to your branch.

@Kurolox
Copy link
Contributor Author

Kurolox commented Oct 25, 2016

Done.

Copy link
Owner

@blackus3r blackus3r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work :)
I will merge it later.

@blackus3r blackus3r merged commit e2d5d27 into blackus3r:swimlane/1.3.0 Oct 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants