Skip to content

Conversation

@drewvolz
Copy link
Member

@drewvolz drewvolz commented Sep 18, 2022

These changes refactor streams list to modernize the data fetching strategy to utilize a useFetch call.

@drewvolz drewvolz force-pushed the drew/streams-use-fetch branch from be89d87 to dccbd7b Compare September 18, 2022 07:06
@drewvolz drewvolz mentioned this pull request Sep 18, 2022
drewvolz and others added 3 commits September 18, 2022 10:38
Co-authored-by: Hawken Rives <hawkrives@fastmail.fm>
Co-authored-by: Hawken Rives <hawkrives@fastmail.fm>
Co-authored-by: Hawken Rives <hawkrives@fastmail.fm>
rye
rye previously requested changes Sep 25, 2022
@drewvolz
Copy link
Member Author

@rye did your questions get answered in this PR?

@drewvolz drewvolz dismissed rye’s stale review November 9, 2022 21:47

Questions addressed

@codecov
Copy link

codecov bot commented Nov 9, 2022

Codecov Report

Merging #6412 (de4714f) into master (4b37aeb) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##           master   #6412      +/-   ##
=========================================
- Coverage    7.88%   7.86%   -0.02%     
=========================================
  Files         289     289              
  Lines        5366    5375       +9     
  Branches     1382    1386       +4     
=========================================
  Hits          423     423              
- Misses       4943    4952       +9     

@drewvolz drewvolz merged commit a82d4c0 into master Nov 9, 2022
@drewvolz drewvolz deleted the drew/streams-use-fetch branch November 9, 2022 22:28
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.

4 participants