Skip to content

Conversation

YuriySamorodov
Copy link
Contributor

It appears parameter MeetingRecordingExpirationDays has been replaced with It appears Set-CsTeamsMeetingPolicy has a new parameter NewMeetingRecordingExpirationDays

image

image

And it keeps the changes:
image

It appears parameter MeetingRecordingExpirationDays has been replaced with It appears Set-CsTeamsMeetingPolicy has a new parameter NewMeetingRecordingExpirationDays
@opbld31
Copy link

opbld31 commented Oct 13, 2021

Docs Build status updates of commit 83d5736:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@yogkumgit
Copy link

@dariomws @get-itips @tseward

Copy link
Contributor

@dariomws dariomws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@YuriySamorodov, while we appreciate your cooperation, there is an internal work in progress to update the parameter on these articles. I think this PR shouldn't be merged. Thank you!

@salarson FYI. Thanks!

@dariomws
Copy link
Contributor

@YuriySamorodov Another important piece of information is that this parameter/setting is not yet available. We cannot remove that note.

@YuriySamorodov
Copy link
Contributor Author

Hi @dariomws , the thing is that parameter NewMeetingRecordingExpirationDays is available already. The screenshots I have provided clearly prove that.

@YuriySamorodov
Copy link
Contributor Author

YuriySamorodov commented Oct 14, 2021

Dear @dariomws ,
Some questions if you don't mind

  1. Should Microsoft customers be able to see the parameter NewMeetingRecordingExpirationDays as they do now?
  2. Set-CsTeamsMeetingPolicy allows NewMeetingRecordingExpirationDays get changed as of now. What does it changes?
  3. Are changes made with NewMeetingRecordingExpirationDays are effective now? (Once again, regular customers can see that parameter and can change it)
  4. Do I understand it right, that NewMeetingRecordingExpirationDays is going to replace MeetingRecordingExpirationDays?

@dariomws
Copy link
Contributor

NewMeetingRecordingExpirationDays and MeetingRecordingExpirationDays are both available in the code but not yet to be set as the note states.

  1. Sometimes you can see parameters that are new and not available yet, and some other parameters that were deprecated and not yet removed from the code.
  2. No impact. It does not apply the change.
  3. No.
  4. I cannot confirm that.

For more information, please see:
https://www.microsoft.com/microsoft-365/roadmap?searchterms=82057&filters=&searchterms=84580
You will be able to see here when this setting is available. It's planned for January 2022.

@YuriySamorodov
Copy link
Contributor Author

YuriySamorodov commented Oct 14, 2021

Thanks @dariomws , appreciate that! Still confused with this because of few things:

  • NewMeetingRecordingExpirationDays looks similar to MeetingRecordingExpirationDays
  • NewMeetingRecordingExpirationDays default value is 60, the same one as for the MeetingRecordingExpirationDays
  • NewMeetingRecordingExpirationDays is exposed to me, regular customer while ```MeetingRecordingExpirationDays```` is not. Is there any way you can confirm with the PG if this is some kind of a bug?
  • NewMeetingRecordingExpirationDays does accept changes

Is there any way you can confirm with PG, if NewMeetingRecordingExpirationDays should be present to the end users as of now? And if yes, what does it change?

@get-itips
Copy link
Contributor

Checking...

@dariomws
Copy link
Contributor

@YuriySamorodov I can see that both parameters appear in the code in the latest version (2.6.0). My apologies but I cannot provide more information.

@YuriySamorodov
Copy link
Contributor Author

Thanks @dariomws , completely understandable. Just to confirm: those are two separate parameters, not aliases, right?

@get-itips
Copy link
Contributor

Some facts
It seems MeetingRecordingExpirationDays is not anymore queryable by the Get-
They seem not to be aliases
MeetingRecordingExpirationDays is in the autocomplete for Set- and New- but I couldn't use it.

@YuriySamorodov
Copy link
Contributor Author

Some facts It seems MeetingRecordingExpirationDays is not anymore queryable by the Get- They seem not to be aliases MeetingRecordingExpirationDays is in the autocomplete for Set- and New- but I couldn't use it.

Thanks @get-itips . Please note though I can run Set-CsTeamsMeetingPolicy -Identity Global -MeetingRecordingExpirationDays -1 no problem and I am an admin of a regular tenant:
image

@get-itips
Copy link
Contributor

the image shows an error when you try to run MeetingRecordingExpirationDays, it says the parameter is not there.

@YuriySamorodov
Copy link
Contributor Author

YuriySamorodov commented Oct 14, 2021

the image shows an error when you try to run MeetingRecordingExpirationDays, it says the parameter is not there.

Right. My intention is to show both MeetingRecordingExpirationDays and NewMeetingRecordingExpirationDays and stress your attention that in fact I can both query and apply changes to NewMeetingRecordingExpirationDays, but not MeetingRecordingExpirationDays

@salarson
Copy link
Contributor

Hey Folks,

NewMeetingRecordingExpirationDays is whitelisted however if run it will not set the expiration of recordings yet (even though it will get populated for R3 tenants). We will announce on Message Center and DOC's when expirations on new recordings will begin.

In regards to the old parameter "MeetingRecordingExpirationDays" - This will be getting removed within the next week .

@get-itips
Copy link
Contributor

@salarson Ok so it's better to hold this PR for now, right?

@salarson
Copy link
Contributor

@salarson Ok so it's better to hold this PR for now, right?

No sir - We want to push this through as the cmdlet is already whitelisted and folks will start seeing it in 2.6.0 build.
We have started to update our DOCs articles also mentioning the new cmdlet .

If anything, we can make a note under this cmdlet stating that although it can be run and will populate in Get , new recordings will not start expiring until a future date.

@get-itips
Copy link
Contributor

@salarson Ok, thanks for clarifying.
@YuriySamorodov Can you add that note? thanks

@dariomws
Copy link
Contributor

@get-itips @YuriySamorodov let's keep the note and reformulate it with this information please.

@YuriySamorodov
Copy link
Contributor Author

@dariomws, not sure if I can make edits in the current PR. Should I submit a new one?

@opbld30
Copy link

opbld30 commented Oct 19, 2021

Docs Build status updates of commit 06ae307:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@dariomws
Copy link
Contributor

@YuriySamorodov you can do it in the same PR. Just click the following button and edit the file:
image

Added note for NewMeetingRecordingExpirationDays letting everyone know it is going to be available early in 2022
@YuriySamorodov
Copy link
Contributor Author

@YuriySamorodov you can do it in the same PR. Just click the following button and edit the file: image

Thanks @dariomws. This is completed. Formatted the note in plain text to be readable in the terminal.

@opbld30
Copy link

opbld30 commented Oct 19, 2021

Docs Build status updates of commit e1f277b:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@dariomws
Copy link
Contributor

Let's apply the same description in the New- cmdlet. I'm proposing this if everyone agree let's apply the same here:
#8484

Made Note for NewMeetingRecordingExpirationDays equal to New-CsTeamsMeetingPolicy
@YuriySamorodov
Copy link
Contributor Author

Let's apply the same description in the New- cmdlet. I'm proposing this if everyone agree let's apply the same here: #8484

Thanks @dariomws ! Replaced note text with the one from #8484

@opbld32
Copy link

opbld32 commented Oct 19, 2021

Docs Build status updates of commit 928eba6:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@opbld32
Copy link

opbld32 commented Oct 19, 2021

Docs Build status updates of commit d3523f5:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Copy link
Contributor

@dariomws dariomws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for your collaboration @YuriySamorodov!

I'm approving but I'll appreciate it if you apply the latest suggestion before merging.

Co-authored-by: Dario Woitasen <33589238+dariomws@users.noreply.github.com>
@opbld30
Copy link

opbld30 commented Oct 20, 2021

Docs Build status updates of commit abc2d6d:

✅ Validation status: passed

File Status Preview URL Details
skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md ✅Succeeded View (skype-ps)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@dariomws
Copy link
Contributor

@YuriySamorodov don't worry my suggestion is applied. You can see them if you go to "Files changed" tab.

Thank you so much for your contribution again!

@yogkumgit yogkumgit assigned tiburd and unassigned ghost , dariomws and get-itips Oct 20, 2021
@yogkumgit
Copy link

@tiburd Please merge this PR. Thanks

@tiburd tiburd merged commit cdb6c65 into MicrosoftDocs:master Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants