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

Change profiles and kink retrieval to the JSON API #74

Open
aliciasprig opened this issue Jun 4, 2014 · 1 comment
Open

Change profiles and kink retrieval to the JSON API #74

aliciasprig opened this issue Jun 4, 2014 · 1 comment

Comments

@aliciasprig
Copy link
Collaborator

The current method of using "PRO" and "KIN" to retrieve profile and kink data directly from the server needs to be depreciated, and replaced with the JSON API https://wiki.f-list.net/Json_endpoints which should be able to do the same and more.

Some thoughts on implementation details (actual implementation need not follow this):

The stub class "FCharacterProfile" should used to store profile data and to possibly perform JSON requests.

The "FServer" class should keep a list/cache of all viewed profiles. "FCharacterProfile" instances should probably keep a reference to the "FServer" instance as the need access to a valid account ticket to retrieve data by the JSON API.

The character profile dialog class should fetch new or cached profiles from the "FServer" instance and receive update notifications from the "FCharacterProfile" instance as data is retrieved.

@kythyria
Copy link
Collaborator

kythyria commented Jun 4, 2014

This would be a more sensible proposal if the current version of the API didn't default to refusing to give that information AT ALL: Not only do you need a ticket, but the character in question needs to belong to an account that has API access enabled.

I'm working on making a trivial wrapper for the API (it does little more than hide the JSON and hide the details of making HTTP requests) as part of revamping the login stuff; it should absorb the change to the v2 API fairly seamlessly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants