MediaWiki integration & integration architecture improvements . #16

Merged
raucao merged 19 commits from feature/6-mediawiki_integration into master 2018-04-21 09:31:30 +00:00
raucao commented 2018-04-19 09:57:36 +00:00 (Migrated from github.com)

closes #6

closes #6
bumi (Migrated from github.com) reviewed 2018-04-19 10:05:52 +00:00
@ -263,3 +136,4 @@
robot.logger.debug(`[hubot-kredits] Proposal created (${proposal.description})`);
// messageRoom(`Let's give ${contributor.name} some kredits for ${proposal.url} (${proposal.description}): https://kredits.kosmos.org`);
});
});
bumi (Migrated from github.com) commented 2018-04-19 10:05:51 +00:00

👍

:+1:
raucao commented 2018-04-19 10:19:02 +00:00 (Migrated from github.com)

This branch works with GitHub now!

This branch works with GitHub now!
bumi (Migrated from github.com) reviewed 2018-04-19 11:38:20 +00:00
bumi (Migrated from github.com) commented 2018-04-19 11:38:20 +00:00

do we need a const util = require('util'); in this file?

do we need a `const util = require('util');` in this file?
raucao (Migrated from github.com) reviewed 2018-04-19 11:40:25 +00:00
raucao (Migrated from github.com) commented 2018-04-19 11:40:25 +00:00

Please ignore the code until I have pushed all of my changes. Thanks.

Please ignore the code until I have pushed all of my changes. Thanks.
raucao commented 2018-04-19 11:41:02 +00:00 (Migrated from github.com)

Correction: it works on my machine, but I have to push some more fixes, when I'm done.

Correction: it works on my machine, but I have to push some more fixes, when I'm done.
raucao commented 2018-04-19 17:21:57 +00:00 (Migrated from github.com)

All done! \o/

All done! \o/
bumi (Migrated from github.com) approved these changes 2018-04-20 11:59:35 +00:00
bumi (Migrated from github.com) left a comment

👍

:+1:
@ -0,0 +21,4 @@
let contrib = contributors.find(c => {
if (typeof c.accounts !== 'object') { return false; }
return c.accounts.find(a => {
a.url === `${process.env.KREDITS_MEDIAWIKI_URL}User:${username}`;
bumi (Migrated from github.com) commented 2018-04-20 11:26:37 +00:00

do we need to do a toLowerCase or something to compare the usernames?

do we need to do a toLowerCase or something to compare the usernames?
@ -0,0 +25,4 @@
});
});
if (!contrib) {
throw new Error();
bumi (Migrated from github.com) commented 2018-04-20 11:27:26 +00:00

could provider an error message here, something like contributor not found ${username}

could provider an error message here, something like `contributor not found ${username}`
raucao (Migrated from github.com) reviewed 2018-04-21 08:16:31 +00:00
@ -0,0 +21,4 @@
let contrib = contributors.find(c => {
if (typeof c.accounts !== 'object') { return false; }
return c.accounts.find(a => {
a.url === `${process.env.KREDITS_MEDIAWIKI_URL}User:${username}`;
raucao (Migrated from github.com) commented 2018-04-21 08:16:31 +00:00

Ideally we have the correct usernames on the blockchain. I haven't looked into the MediaWiki specifics there.

Ideally we have the correct usernames on the blockchain. I haven't looked into the MediaWiki specifics there.
bumi (Migrated from github.com) reviewed 2018-04-21 09:28:54 +00:00
@ -0,0 +21,4 @@
let contrib = contributors.find(c => {
if (typeof c.accounts !== 'object') { return false; }
return c.accounts.find(a => {
a.url === `${process.env.KREDITS_MEDIAWIKI_URL}User:${username}`;
bumi (Migrated from github.com) commented 2018-04-21 09:28:54 +00:00

we only have the usernames in the IPFS profile. which actually is fine because we want to store as little as possible in the contract

we only have the usernames in the IPFS profile. which actually is fine because we want to store as little as possible in the contract
raucao added the
kredits-2
label 2025-01-23 21:25:36 +00:00
Sign in to join this conversation.
No description provided.