iTwin / itwinjs-core

Monorepo for iTwin.js Library
https://www.itwinjs.org
MIT License
606 stars 210 forks source link

Deferred cesiumIonKey initialization #6520

Open FelixGirard opened 6 months ago

FelixGirard commented 6 months ago

Is your feature request related to a problem? Please describe. Since we use a service (PSS) to get the key, we need to do it after we are authenticated.

Describe the solution you'd like I would like to have a deferred way to set the cesiumIonKey.

Describe alternatives you've considered

Additional context Seems like you embrace the deferred initialization for other properties: https://github.com/iTwin/itwinjs-core/pull/6449 So I think it would make sense to apply that to others.

ben-polinsky commented 6 months ago

This makes sense. We'll get someone on it.

pmconne commented 6 months ago

Most of the context for this issue is in the issue linked by @FelixGirard, so let's discuss it there.

tcobbs-bentley commented 6 months ago

Based on https://github.com/iTwin/itwinjs-core/pull/6526, should this issue going to be closed as not planned?