- Issue created by @kaustubhb
- 🇮🇳India kaustubhb Mumbai
I have upgraded the apis from windows AZURE AD to Microsoft Graph, I also have accommodated the two issues mentioned above as part of single patch to reduce effort and better maintainability.
- 🇮🇳India kaustubhb Mumbai
Adding a new patch with a new configuration field to enter the base endpoint url for the Graph API and ease the upgrade of api in future
- 🇮🇳India kaustubhb Mumbai
After reviewing the patch updated few things like default value of the new field and sone change in code standards as per drupal.
- 🇮🇳India kaustubhb Mumbai
I found one more issue which deals with the pagination of the groups api. Gorups api serves 100 records per page so in groups mapping if the group is on page no 101 and so on the respective Drupal role is removed from the Drupal user due to lack of group information and user looses the group
Issue link
https://www.drupal.org/project/openid_connect_windows_aad/issues/3446321 🐛 Handling pagination for user groups ActiveUpdating the patch accordingly to accommodate the pagination in the groups api to fix the roles
- 🇮🇳India kaustubhb Mumbai
patch number #8 ✨ [D7] Consolidate api upgrades from Azure AD to Microsoft Graph Active had issue with manual mapping, The manual mapping was not workings as expected if there is only one page of groups.