Skip to content
This repository has been archived by the owner on Jul 9, 2022. It is now read-only.

Messenger Day implementation #434

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

Conversation

gamelaster
Copy link
Contributor

@gamelaster gamelaster commented Mar 29, 2017

  • Make executing function for Batch graphQL queries
  • Make working Messenger Days query
  • Format and parse Messenger Days Images
  • Format and parse Messenger Days Videos
  • Documentation

With that graph QL batch ability you can fetch almost everything, it's really powerful.
The syntax of graphQL is little chaotic, but after some time of investigating, you get it!
By the way, in Messenger Days has been an "Sticker Feature", but probably unused!

@Schmavery
Copy link
Owner

@gamelaster is on fire 🔥 🔥

@gamelaster
Copy link
Contributor Author

@Schmavery Could you review?

@Schmavery
Copy link
Owner

Now that we're getting into some of the graphql interface, would it make any sense to find a supported graphql client library that we can use rather than reimplementing all the logic ourselves?

I don't know much about it, but a quick search revealed a couple of options, including https://github.com/kadirahq/lokka and https://github.com/gucheen/FetchQL
@bsansouci

@gamelaster
Copy link
Contributor Author

@Schmavery As far as I search, I didn't found an library what returns similar code like going into Facebook's graphQL

@gamelaster
Copy link
Contributor Author

I found some things about graphQL and QueryFragments, but still I can't found any similar syntax close to this.
BTW. On what is stuck this PR? 😆

@Schmavery
Copy link
Owner

Sorry @gamelaster, just haven't had much time to think about what is the best way to move forward here. Maybe @bsansouci has some ideas. I kind of wish there was a way that we didn't need to have src/executeGraphQueryBatch.js

@gamelaster
Copy link
Contributor Author

@Schmavery executeGraphQueryBatch is not of course needed, I just wanted to let users ability to execute custom graphQL queries, it's really powerful. With that you can fetch the normal messages etc (It's just require experimentation). We can move this function to utils.js for sample 😊

@ravkr
Copy link
Contributor

ravkr commented Apr 19, 2017

@gamelaster what if I want to get days of all friends? do I have to call api.getUserDay for every user in my friends?

@gamelaster
Copy link
Contributor Author

@ravkr yes, for every user, but you now gave me idea, where I can try modify graphQL batch to fetch all users. It's could work, but not sure.

@gamelaster
Copy link
Contributor Author

After testing, it's not working anymore, I must investigate it again.

@larryqiann
Copy link

Did the API on Facebook's end change again?

@bsansouci
Copy link
Collaborator

Heyyyy @gamelaster are you still working on this?

@gamelaster
Copy link
Contributor Author

@bsansouci sorry for late response, I lost notification for this. Well, the actual code in merge request is not working anymore, the format of graphQL has been changed. I will try to make a new one (probably I will close this PR and make new one, since I saw some code already integrated for graphQL)

ravkr referenced this pull request in ravkr/facebook-chat-api Mar 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants