Skip to content

Conversation

@Yashasvi-JP
Copy link
Collaborator

No description provided.

@Yashasvi-JP Yashasvi-JP changed the title add mgci check for moto migration:add mgci check for moto Jul 14, 2025
@Yashasvi-JP Yashasvi-JP changed the title migration:add mgci check for moto migration: add mgci check for moto Jul 14, 2025
Comment on lines +1361 to +1378
let motoGcis =
match mAuthType {
Some(authType) => gcis
.iter()
.filter(
|gci|
match &gci.authType {
Some(gciAuthType) => gciAuthType == &auth_type_to_text(&authType),
None => false
})
.collect(),
None => vec![]
};

let gciIds = motoGcis
.iter()
.map(|gci| gci.id.clone())
.collect::<Vec<_>>();
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
let motoGcis =
match mAuthType {
Some(authType) => gcis
.iter()
.filter(
|gci|
match &gci.authType {
Some(gciAuthType) => gciAuthType == &auth_type_to_text(&authType),
None => false
})
.collect(),
None => vec![]
};
let gciIds = motoGcis
.iter()
.map(|gci| gci.id.clone())
.collect::<Vec<_>>();
let gciIds =
match mAuthType {
Some(authType) => gcis
.iter()
.filter(
|gci|
match &gci.authType {
Some(gciAuthType) => gciAuthType == &auth_type_to_text(&authType),
None => false
})
.map(|gci| gci.id.clone())
.collect(),
None => vec![]
};

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.

4 participants