Skip to content

Conversation

@maxy-shpfy
Copy link
Collaborator

@maxy-shpfy maxy-shpfy commented Dec 16, 2025

Description

Removed the deprecated checkIfFavorited method from the ComponentLibraryProvider. This includes:

  • Removing the method implementation from the provider
  • Removing the method from the context type definition
  • Removing related tests
  • Updating test mocks that referenced this method

Type of Change

  • Cleanup/Refactor
  • Breaking change

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Test Instructions

Ensure that no components are using the deprecated checkIfFavorited method. All components should be updated to use alternative methods for checking if a component is favorited.

@maxy-shpfy maxy-shpfy force-pushed the 12-15-refactor_remove_checkiffavorited_from_componentlibraryprovider branch from 36bc459 to db9e4b3 Compare December 17, 2025 19:05
@maxy-shpfy maxy-shpfy force-pushed the 12-15-feat_lazy_load_of_the_yaml_library_components branch 2 times, most recently from b2e6655 to 838ef5c Compare December 17, 2025 19:16
@maxy-shpfy maxy-shpfy force-pushed the 12-15-refactor_remove_checkiffavorited_from_componentlibraryprovider branch 2 times, most recently from 66e347b to b62d6e8 Compare December 17, 2025 19:21
@maxy-shpfy maxy-shpfy force-pushed the 12-15-feat_lazy_load_of_the_yaml_library_components branch from 838ef5c to 1d63324 Compare December 17, 2025 19:21
@maxy-shpfy maxy-shpfy force-pushed the 12-15-feat_lazy_load_of_the_yaml_library_components branch from 1d63324 to 26f27d1 Compare December 18, 2025 16:54
@maxy-shpfy maxy-shpfy requested a review from a team as a code owner December 18, 2025 16:54
@maxy-shpfy maxy-shpfy force-pushed the 12-15-feat_lazy_load_of_the_yaml_library_components branch from 26f27d1 to 43b189e Compare December 18, 2025 16:55
@maxy-shpfy maxy-shpfy force-pushed the 12-15-refactor_remove_checkiffavorited_from_componentlibraryprovider branch from b62d6e8 to be7a59d Compare December 18, 2025 16:55
@maxy-shpfy maxy-shpfy changed the base branch from 12-15-feat_lazy_load_of_the_yaml_library_components to graphite-base/1559 December 18, 2025 17:51
@maxy-shpfy maxy-shpfy force-pushed the 12-15-refactor_remove_checkiffavorited_from_componentlibraryprovider branch from be7a59d to 68c4190 Compare December 18, 2025 17:52
@graphite-app graphite-app bot changed the base branch from graphite-base/1559 to master December 18, 2025 17:52
@maxy-shpfy maxy-shpfy force-pushed the 12-15-refactor_remove_checkiffavorited_from_componentlibraryprovider branch from 68c4190 to 59ef157 Compare December 18, 2025 17:53
Copy link
Collaborator Author

maxy-shpfy commented Dec 18, 2025

Merge activity

  • Dec 18, 5:59 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Dec 18, 5:59 PM UTC: @maxy-shpfy merged this pull request with Graphite.

@maxy-shpfy maxy-shpfy merged commit b856ef0 into master Dec 18, 2025
7 of 8 checks passed
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.

3 participants