bug: `ModalOptions` no longer accepts a Type parameter

[Ionaru]

Prerequisites

• I have read the Contributing Guidelines.
• I agree to follow the Code of Conduct.
• I have searched for existing issues that already report this problem, without success.

Ionic Framework Version

v8.x

Current Behavior

Using this code:

import { ModalOptions } from '@ionic/angular';

const modal = await this.modalController.create({
  component: MyModalComponent,
} satisfies ModalOptions<typeof MyModalComponent>);

It errors:

[ERROR] TS2315: Type 'ModalOptions' is not generic.

Expected Behavior

This code:

import { ModalOptions } from '@ionic/angular';

const modal = await this.modalController.create({
  component: MyModalComponent,
} satisfies ModalOptions<typeof MyModalComponent>);

Should not error and correctly verify that component is MyModalComponent

Steps to Reproduce

N/A

Code Reproduction URL

N/A

Ionic Info

N/A

Additional Information

Bug introduced in #30899, specifically in commit 641f319.

Ionic 8.8.0 was the first version with this bug.

[ionitron-bot]

Thanks for the issue! This issue has been labeled as needs reproduction. This label is added to issues that need a code reproduction.

Please reproduce this issue in an Ionic starter application and provide a way for us to access it (GitHub repo, StackBlitz, etc). Without a reliable code reproduction, it is unlikely we will be able to resolve the issue, leading to it being closed.

If you have already provided a code snippet and are seeing this message, it is likely that the code snippet was not enough for our team to reproduce the issue.

For a guide on how to create a good reproduction, see our Contributing Guide.

[Ionaru]

@gnbm Repro: https://stackblitz.com/edit/y5pgfd6n?file=src%2Fapp%2Fexample.component.ts

This code used to work in Ionic 8.7: https://stackblitz.com/edit/y5pgfd6n-w6qegpsp?file=src%2Fapp%2Fexample.component.ts

[ShaneK]

Hey, sorry! This issue was created in this PR when we at the last minute decided to re-export ModalOptions and accidentally overlooked this problem. I'll have a PR up to fix this shortly and hopefully get this out in the next patch version!

[ShaneK]

Hey! This was fixed in today's release, v8.8.2! We appreciate the report, sorry about the issue!

[ionitron-bot]

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.