Quote from: jernau on January 30, 2014, 07:46:35 PM

Quote from: lakerta06 on January 30, 2014, 07:26:01 PM

Quote from: lakerta06 on January 30, 2014, 03:16:13 PM
Here is a new one, but I am not sure which way to go:

On page http://invictus.io/bitshares-ags.php the term "Allied developers" is seen twice. I think both "a" and "d" shoud be in lower case. Or if "Allied Developers" is a proper noun (which I do not think so), then it should be both uppercase.

So I went the lowercase way and created a pull request:

https://github.com/super3/invictus.io/pull/78

Let me know if it should be the other way around.

rejected with no comment  ???

Guess it was a style choice to have Allied developers.

Lol  :)

Quote from: lakerta06 on January 30, 2014, 07:26:01 PM

Quote from: lakerta06 on January 30, 2014, 03:16:13 PM
Here is a new one, but I am not sure which way to go:

On page http://invictus.io/bitshares-ags.php the term "Allied developers" is seen twice. I think both "a" and "d" shoud be in lower case. Or if "Allied Developers" is a proper noun (which I do not think so), then it should be both uppercase.

So I went the lowercase way and created a pull request:

https://github.com/super3/invictus.io/pull/78

Let me know if it should be the other way around.

rejected with no comment  ???

Guess it was a style choice to have Allied developers.

Quote from: lakerta06 on January 30, 2014, 03:16:13 PM
Here is a new one, but I am not sure which way to go:

On page http://invictus.io/bitshares-ags.php the term "Allied developers" is seen twice. I think both "a" and "d" shoud be in lower case. Or if "Allied Developers" is a proper noun (which I do not think so), then it should be both uppercase.

So I went the lowercase way and created a pull request:

https://github.com/super3/invictus.io/pull/78

Let me know if it should be the other way around.

rejected with no comment  ???

I've made two changes in this pull request: https://github.com/super3/invictus.io/pull/80 (username themanifold). One is a grammar typo, the other is to create some favicons. Everyone loves favicons.

Here is a new one, but I am not sure which way to go:

On page http://invictus.io/bitshares-ags.php the term "Allied developers" is seen twice. I think both "a" and "d" shoud be in lower case. Or if "Allied Developers" is a proper noun (which I do not think so), then it should be both uppercase.

So I went the lowercase way and created a pull request:

https://github.com/super3/invictus.io/pull/78

Let me know if it should be the other way around.

Quote from: lakerta06 on January 28, 2014, 08:21:15 PM
"each" should be followed by a singular noun:

https://github.com/super3/invictus.io/pull/75

Paid, thanks

Quote from: Troglodactyl on January 28, 2014, 03:28:30 AM

https://github.com/super3/invictus.io/pull/72

Done.  Please credit phabit for this one.

Thanks - very generous! I'll learn to use github when I get a spare moment.

Thanks, what a nice feeling!

Till the pull request #74, I am going to pay as follows:

xeroc       PdbNaeWMsB2JXWWURwxVNA2jdZEmpuUaME 1
gigawatt           Pstkk1gZCxc4GEwS1eBAykYwVmcubU1P8L 9
jernau      PgiEykg2RATYwWYhFtyNRqwSxQyEApLSmW 12
lakerta06           PeyQbhgPJra5Cts7sEXN5ccQoTsLtGxFT8 4
toast      PtoastsCnAW4JfoSFbjvZjSBRPUB9qBoSU 1
Troglodactyl   Pt2FcsBjnnbjAuVLghytyYmeKyqykVZhf5 8
phabit              PoPcJAzDM5WMnTfBLuMpXSpbW8hjPV1c7d 1
38PTSWarrior   PmJwbQBFssGK1yd1GLQb8hHswzyRkqqNzB 1
nametooshort   PvDZqsSyAsCDYNyYCfwZmy19EVohxnbnKB 1
todofixthis   PZ32Dv8NA1CBNtYyu5Z14qWNL6VWDSP4HC 16

todofixthis, you changed lots of inconsistency, I will count your total contribution as 16 PTS. Please let me know if you think it is fair and also PM me your address.
currencydebt, could you please PM me your github id or pull request number.

It is hard to match forum ID and github ID. If your pull request #<=74 and got merged but not listed above, please PM me.

"each" should be followed by a singular noun:

https://github.com/super3/invictus.io/pull/75

Quote from: todofixthis on January 28, 2014, 05:15:36 AM

• https://github.com/super3/invictus.io/pull/71 (merged)
• https://github.com/super3/invictus.io/pull/73 (merged)

PTS addy is in the pull request description.

Oops... I really hope I get credit for each typo I fixed and not just 1 PTS per pull request! /:

Sure thing, you get one per typo found/fixed.  I am going to delegate the task of paying these bounties out to Amazon. 

Thanks for everyone who has worked on this while we were busy in Miami.

Quote from: nametooshort on January 27, 2014, 08:03:02 AM
https://github.com/super3/invictus.io/pull/69

https://github.com/super3/invictus.io/pull/74
pull/69 is unable to merge...

• https://github.com/super3/invictus.io/pull/71 (merged)
• https://github.com/super3/invictus.io/pull/73 (merged)

PTS addy is in the pull request description.

Oops... I really hope I get credit for each typo I fixed and not just 1 PTS per pull request! /:

Quote from: phabit on January 27, 2014, 11:29:30 PM
Hi all,
Don't have the time to work out how to use github right now, but just spotted the link on http://invictus.io/bitshares-pts.php to the beeeer mining pool isn't correct - it's points to http://pts.beeeeer.org/, but should be http://ptsweb.beeeeer.org.

Someone who knows how to do it, feel free to submit a fix, and share the bounty with me (PoPcJAzDM5WMnTfBLuMpXSpbW8hjPV1c7d)!  :D

https://github.com/super3/invictus.io/pull/72

Done.  Please credit phabit for this one.

Hi all,
Don't have the time to work out how to use github right now, but just spotted the link on http://invictus.io/bitshares-pts.php to the beeeer mining pool isn't correct - it's points to http://pts.beeeeer.org/, but should be http://ptsweb.beeeeer.org.

Someone who knows how to do it, feel free to submit a fix, and share the bounty with me (PoPcJAzDM5WMnTfBLuMpXSpbW8hjPV1c7d)!  :D