Skip to content

Add new condition to DL2365Groupupgrade script#693

Merged
bill-long merged 2 commits into
microsoft:mainfrom
hazemembaby:haembab-addnewcondition
Aug 15, 2021
Merged

Add new condition to DL2365Groupupgrade script#693
bill-long merged 2 commits into
microsoft:mainfrom
hazemembaby:haembab-addnewcondition

Conversation

@hazemembaby
Copy link
Copy Markdown
Contributor

Issue:
Describe what the issue is you are addressing

  • I've added new condition to test DL owners with RecipientTypeDetails other than UserMailbox, MailUser
  • I've modified the EAP condition to work case sensitively while comparing DL PrimarySmtpAddress& EnabledPrimarySMTPAddressTemplate
  • Modified readme file by change title O365 to Microsoft 365

Reason:
Describe what the reason is for making the change

To address these new conditions I've mentioned earlier that might fail DL to M365 group upgrade

Fix:
Short description of the fix

  • I've added new condition to test DL owners with RecipientTypeDetails other than UserMailbox, MailUser
  • I've modified the EAP condition to work case sensitively while comparing DL PrimarySmtpAddress& EnabledPrimarySMTPAddressTemplate
  • Modified readme file by change title O365 to Microsoft 365

Validation:
Provide if applicable
image

Comment thread M365/src/DLT365Groupsupgrade.ps1 Outdated
Comment thread M365/src/DLT365Groupsupgrade.ps1 Outdated
@bill-long bill-long merged commit 287fa1e into microsoft:main Aug 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants