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.

Sign up for GitHub

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

Jump to bottom

Recent changes #148

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

Recent changes #148

wants to merge 26 commits into from

Conversation

mardam
Copy link
Contributor

@mardam mardam commented Jun 14, 2015

This solution fetches and parses recent changes from the RSS feed.

mardam added 21 commits May 29, 2015 18:46
@mardam
Created RecentChangesFetcher
8772892
@mardam
add first implementation of getRecentChanges(); returns a set of Strings
8a37efb 
of the property names
@mardam
first jUnit-Test with MockObject added
85e7724
@mardam
add parseTime() and corresponding test
f37e9cb
@mardam
add parseAuthor and corresponding test
a673fdf
@mardam
change datatype of testfile
85a57eb
@mardam
replace Scanner in parsing of RSS feed by BufferedReader
24c8e7c
@mardam
remove unnecessary public visabilites
0cff4a1
@mardam
Edit time parsing to fix positions in line
4e5506c
@mardam
Edit parsing of author to a fix beginning
d95069e
@mardam
created class RecentChange
858ade7
@mardam
Fix for testParseDate()
c4c32d3
@mardam
Put RecentChange to own file and removed as inner class from
7198237 
RecentChangesFetcher
@mardam
add Constructor to RecentChange
a244f0f
@mardam
added RecentChangesTest
78ff331
@mardam
add test for Comparable-interface of RecentChange
5538726
@mardam
expand test for equal RecentChange
b22ca40
@mardam
correct style guide
250a451
@mardam
fix of API URL
c8ced9e
@mardam
add buildUrl
fd6fa61
@mardam
Overload getRecentChanges
2fb42cf
@mkroetzsch mkroetzsch self-assigned this Jun 17, 2015
@mkroetzsch
Copy link
Member

This should be updated to use the new ApiConnection before being merged.

wetneb
wetneb reviewed Feb 3, 2021
View reviewed changes
Copy link
Member

@wetneb wetneb left a comment

Choose a reason for hiding this comment

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

I could imagine this being still useful: parsing the recent changes is still the only way one can stay in sync with a Wikibase instance, out of the box.

I would be especially interested in some logic for recent changes polling (generating a real-time stream of events, where each event appears in order, exactly once). This PR does not do this yet, but I imagine the classes it introduces could be a good basis for that.

If there is interest I could try to take over this PR and get it merged. Thoughts? @Tpt @robertvazan

@robertvazan
Copy link
Collaborator

@wetneb I will definitely need to read recent changes in my project at some point in the future. I would certainly test this and maybe I would find something to contribute.

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

@wetneb wetneb wetneb left review comments

Assignees

@mkroetzsch mkroetzsch

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@mardam @mkroetzsch @robertvazan @wetneb @monkey2000