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

Sign in process doesn't allow for user input #955

Closed
AndreeaCristinaRadacina opened this issue May 8, 2024 · 9 comments · Fixed by #963
Closed

Sign in process doesn't allow for user input #955

AndreeaCristinaRadacina opened this issue May 8, 2024 · 9 comments · Fixed by #963
Assignees
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer. medium (1d) - This label is used for issues that can be completed within 1 day or less. Priority-Medium Expected resolution time - up to 1 month.

Comments

@AndreeaCristinaRadacina
Copy link

Description

A user reported that he can no longer choose the Facebook profile that should be linked to his WP instance.

Step-by-step reproduction instructions

  1. Go to ROP
  2. Try to connect to Facebook

The connection will be made, but you cannot input the profile or choose which pages to appear, as it was before.

Screenshots, screen recording, code snippet or Help Scout ticket

Once you click on sign in, this will appear:
image

ref:https://secure.helpscout.net/conversation/2590687579/418114?viewId=212385

Environment info

No response

Is the issue you are reporting a regression

Yes, this is a regression.

@AndreeaCristinaRadacina AndreeaCristinaRadacina added the bug This label could be used to identify issues that are caused by a defect in the product. label May 8, 2024
@pirate-bot pirate-bot added customer report Indicates the request came from a customer. regression Issue represents a change in behavior from a previous version that is not intended or desired.. labels May 8, 2024
@vytisbulkevicius vytisbulkevicius added Priority-Medium Expected resolution time - up to 1 month. and removed regression Issue represents a change in behavior from a previous version that is not intended or desired.. labels May 9, 2024
@AndreeaCristinaRadacina
Copy link
Author

@Soare-Robert-Daniel
Copy link
Contributor

This was caused by the re-authorization process, which tries to log in to the previously saved users. One quick way for the user to choose the profile is to use incognito mode so that Facebook does not reuse the cookies.

@AndreeaCristinaRadacina
Copy link
Author

@Soare-Robert-Daniel I shared this solution, but the answer is not positive regarding the result.

@Soare-Robert-Daniel
Copy link
Contributor

Besides this, we should also update the Facebook API version since Facebook is starting to move on:

'default_graph_version' => 'v12.0',

https://developers.facebook.com/docs/graph-api/changelog

@Soare-Robert-Daniel Soare-Robert-Daniel added the medium (1d) - This label is used for issues that can be completed within 1 day or less. label May 29, 2024
@Soare-Robert-Daniel Soare-Robert-Daniel linked a pull request May 29, 2024 that will close this issue
@Soare-Robert-Daniel
Copy link
Contributor

@AndreeaCristinaRadacina, I have reverted to the old behavior for authentication. You can keep an eye on if the users still have this difficulty.

@AndreeaCristinaRadacina
Copy link
Author

As discussed, the login is possible, but the whole modal with permissions and page selection is gone.

@Soare-Robert-Daniel
Copy link
Contributor

@AndreeaCristinaRadacina, we can not do too much on our part. Once permission has been granted, Facebook will not ask again (also mentioned in their docs).

Based on this, a possible solution for the users to re-add the asking stage is to remove the post permission in their Facebook dashboard, as shown in the video below.

If they have multiple websites, this approach might be problematic since if they do this for one website, the login on other websites might be revoked since the access key will be invalid.

Image

2024-05-30_11-59-20.mp4

@Soare-Robert-Daniel
Copy link
Contributor

@AndreeaCristinaRadacina, I tweaked the login process again, and the account selection is now showing on my end 🙏

@AndreeaCristinaRadacina
Copy link
Author

@Soare-Robert-Daniel indeed, it works now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer. medium (1d) - This label is used for issues that can be completed within 1 day or less. Priority-Medium Expected resolution time - up to 1 month.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants