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

[Logs UI] Replace references to IFieldType #107885

Closed
weltenwort opened this issue Aug 9, 2021 · 2 comments · Fixed by #121108
Closed

[Logs UI] Replace references to IFieldType #107885

weltenwort opened this issue Aug 9, 2021 · 2 comments · Fixed by #121108
Assignees
Labels
chore Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v8.0.0

Comments

@weltenwort
Copy link
Member

weltenwort commented Aug 9, 2021

📝 Summary

The IFieldType interface is going to be removed because it's too ambiguous. We should replace its usage with either IndexPatternField or IndexPatternFieldBase.

part of #107235

✔️ Acceptance criteria

  • No references to IFieldType exist in the Logs UI codebase.
  • The impacted code remains fully typed using appropriate replacements.
@weltenwort weltenwort added Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v7.16.0 labels Aug 9, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@Kerry350
Copy link
Contributor

Kerry350 commented Dec 8, 2021

Picking this up as it follows on from #120762 nicely and means those chores are out of the way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v8.0.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants