Page MenuHomePhabricator
Create Task
Maniphest T130035

Get Cyberbot to use logging API
Closed, ResolvedPublic5 Estimated Story Points
Actions

Description

Now that the deallink logging API is finished, we should have Cyberbot start using it.

Documentation for logging API:
https://meta.wikimedia.org/wiki/Fixing_dead_links/Deadlink_logging_app

Related Objects
Search...

Event Timeline

kaldari created this task.Mar 15 2016, 5:39 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 15 2016, 5:39 PM
kaldari triaged this task as Medium priority.Mar 15 2016, 5:39 PM
kaldari moved this task from New & TBD Tickets to Up Next (June 3-21) on the Community-Tech board.
kaldari added a subscriber: Cyberpower678.
• DannyH edited projects, added Community-Tech-Sprint; removed Community-Tech.Mar 15 2016, 5:41 PM
• DannyH set the point value for this task to 5.
kaldari added a comment.Mar 23 2016, 5:11 AM

@Cyberpower678: Do you have any thoughts or concerns about this task? Is this something that you would like to work on or would it be better for the Community Tech team to do the integration?

Cyberpower678 added a comment.Mar 23 2016, 1:27 PM
In T130035#2143452, @kaldari wrote:

@Cyberpower678: Do you have any thoughts or concerns about this task? Is this something that you would like to work on or would it be better for the Community Tech team to do the integration?

I don't mind doing it. I just need to get around to it. If you want to do, I don't mind either. I would suggest the actual calls are made in the API class and it would be called from the analyzePage function.

• DannyH raised the priority of this task from Medium to High.Mar 24 2016, 5:20 PM
Cyberpower678 claimed this task.Mar 24 2016, 5:47 PM
Cyberpower678 lowered the priority of this task from High to Medium.
Cyberpower678 added a comment.Mar 25 2016, 1:00 AM
In T130035#2143452, @kaldari wrote:

@Cyberpower678: Do you have any thoughts or concerns about this task? Is this something that you would like to work on or would it be better for the Community Tech team to do the integration?

So the bot does not count links not fixed. It would need to take a bit to get that implemented the way the bot is built.

Cyberpower678 added a comment.Mar 25 2016, 2:03 AM

I don't have the password to this.

Cyberpower678 closed this task as Resolved.Mar 25 2016, 11:12 PM
Niharika subscribed.Mar 26 2016, 3:01 AM
In T130035#2150276, @Cyberpower678 wrote:
In T130035#2143452, @kaldari wrote:

@Cyberpower678: Do you have any thoughts or concerns about this task? Is this something that you would like to work on or would it be better for the Community Tech team to do the integration?

So the bot does not count links not fixed. It would need to take a bit to get that implemented the way the bot is built.

Could you not just send (total number of links - number of links fixed)?

Cyberpower678 added a comment.EditedMar 26 2016, 3:03 AM
In T130035#2152712, @Niharika wrote:
In T130035#2150276, @Cyberpower678 wrote:
In T130035#2143452, @kaldari wrote:

@Cyberpower678: Do you have any thoughts or concerns about this task? Is this something that you would like to work on or would it be better for the Community Tech team to do the integration?

So the bot does not count links not fixed. It would need to take a bit to get that implemented the way the bot is built.

Could you not just send (total number of links - number of links fixed)?

I already fixed this. I just added a counter to the part of the code that tags or ignores instead of fixes.

Niharika added a comment.Mar 26 2016, 3:04 AM
In T130035#2152714, @Cyberpower678 wrote:
In T130035#2152712, @Niharika wrote:
In T130035#2150276, @Cyberpower678 wrote:
In T130035#2143452, @kaldari wrote:

@Cyberpower678: Do you have any thoughts or concerns about this task? Is this something that you would like to work on or would it be better for the Community Tech team to do the integration?

So the bot does not count links not fixed. It would need to take a bit to get that implemented the way the bot is built.

Could you not just send (total number of links - number of links fixed)?

Huh?

Number of links not fixed = Total number of reference links on the page - Number of links fixed.

kaldari added a comment.Mar 26 2016, 5:58 AM

Actually, numn is supposed to be for dead links that are not fixed, not any links that are not fixed. Knowing how many dead links are not successfully fixed is much more useful information for the Internet Archive.

Niharika added a comment.Mar 26 2016, 9:32 AM
In T130035#2152779, @kaldari wrote:

Actually, numn is supposed to be for dead links that are not fixed, not any links that are not fixed. Knowing how many dead links are not successfully fixed is much more useful information for the Internet Archive.

Yes, I talked to Max about this and he says that the bot is correctly logging this number to the logger already.

• DannyH moved this task from Ready to Q1 2018-19 on the Community-Tech-Sprint board.Mar 28 2016, 5:18 PM
• DannyH added a parent task: T125610: Epic: Create a centralized logging interface for tracking and reporting dead link fixes.Mar 28 2016, 10:24 PM
• DannyH edited projects, added Community-Tech; removed Community-Tech-Sprint.Mar 28 2016, 10:31 PM
• DannyH moved this task from Up Next (June 3-21) to Archive on the Community-Tech board.Mar 28 2016, 10:35 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits