Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Audit experimental features in ET/event-countdown-block #46519

Closed
1 task
WunderBart opened this issue Oct 16, 2020 · 4 comments
Closed
1 task

Audit experimental features in ET/event-countdown-block #46519

WunderBart opened this issue Oct 16, 2020 · 4 comments
Labels
[Goal] Gutenberg Working towards full integration with Gutenberg [Type] Task

Comments

@WunderBart
Copy link
Member

Master tracking issue: #44251
ET tracking issue: #46489

event-countdown-block is dependant on experimental features without a fallback for a scenario where such a feature is not supported anymore. Below is a list of usages that we need to update in this matter:

@WunderBart WunderBart added the [Goal] Gutenberg Working towards full integration with Gutenberg label Oct 16, 2020
@WunderBart
Copy link
Member Author

cc @mkaz

@mkaz
Copy link
Contributor

mkaz commented Oct 17, 2020

The experimental get settings in wp-date is a relatively simple use. I'm not quite sure why it was marked experimental in the first place, or why it required a function to return a variable set in the date package.

The code is basically the following:

let settings = {  /* ... bunch of settings ... */  };
export function __experimentalGetSettings() { return settings; }

There are numerous ways to solve, and would depend on when/how the experimental change is made. Simply exporting the settings from the package and doing an import and check would work.

Related, @vcanales is reworking the date package, so another option would be to implement the new functions when updated.

@vcanales
Copy link
Member

Related, @vcanales is reworking the date package, so another option would be to implement the new functions when updated.

This seems very reasonable. As you point out, the function itself returns the settings object and not much else.

On the other hand, when it was marked as experimental, the intention was to eventually deprecate it, replacing it with a settings endpoint. Here's the origin of the discussion around deprecating getSettings. I don't know if any work has been done around implementing the date settings API mentioned in that discussion.

@sirreal
Copy link
Member

sirreal commented Nov 27, 2020

Closing this as of #47712 / p4TIVU-9xW-p2

@sirreal sirreal closed this as completed Nov 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Goal] Gutenberg Working towards full integration with Gutenberg [Type] Task
Projects
None yet
Development

No branches or pull requests

5 participants