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

Facebook-Connector - Notifications #33

Merged
merged 6 commits into from
Apr 1, 2012
Merged

Facebook-Connector - Notifications #33

merged 6 commits into from
Apr 1, 2012

Conversation

CatoTH
Copy link
Collaborator

@CatoTH CatoTH commented Mar 31, 2012

  • The "your session became invalid as you changed your password"-notification is sent once every 3 days at max.
  • Instead of a broken image in this notification, a fallback-avatar is displayed.
  • The "Problem with Real-Time Updates"-Mail now contains a link to the plugin where the administrator can try to re-activate it.

…BUG: if a connected FB-App is deleted and a new FB-App is connected, the app_access_token of the former App remained in the config and made it impossible to activate Real-Time Updates with the new App. USABILITY: Now the user's connector settings page automatically recognizes if the connection is still valid and acts accordingly. SECURITY: XSRF-Protection.
@CatoTH
Copy link
Collaborator Author

CatoTH commented Apr 1, 2012

I just extended the Pull request by one more commit:

[FEATURE] AppID/Secret can now be entered on the plugin's admin page.
[BUG] if a connected FB-App is deleted and a new FB-App is connected, the app_access_token of the former App remained in the config and made it impossible to activate Real-Time Updates with the new App.
[USABILITY] Now the user's connector settings page automatically recognizes if the connection is still valid and acts accordingly.
[SECURITY] XSRF-Protection.

@hurricanerainbow
Copy link
Collaborator

looks good dude

friendica pushed a commit that referenced this pull request Apr 1, 2012
Facebook-Connector - Notifications
@friendica friendica merged commit c739f47 into friendica:master Apr 1, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants