#39 [OutReachy] Added Facebook Tagged Updates of @TheFedoraProject
Closed by a2batic. Opened by twishi.
twishi/Fedora-app master  into  master

Download 39.patch

Fixed #8
Added Facebook tagged updates from the Facebook page https://www.facebook.com/TheFedoraProject

Before merging,
Please update line 6 (app access token which has access to the facebook page, TheFedoraProject) in src/providers/fb/fb.ts

Could you rewrite this logic, so that the entire code can be reused for fetching updates from any page?

I see a lot of repetition here, which is not good.

console.log? Consider a more appropriate error notification.

DRY again :)

Will do that.

@twishi, the current function for fetching facebook posts has to be replaced, instead of adding another FBCONFIG.

@twishi any updates on this ?

2 new commits added

  • Added Method of Fetching Popular hastags on Fedora Twitter Handle
  • Added Method of Fetching Tweets

@twishi can you please send a separate PR for twitter ?

Closing it.

Pull-Request has been closed by a2batic

Metadata