Skip to content

Fix eternal loading progress #719 #697 - #738

Open
QArtur99 wants to merge 2 commits into
nickbutcher:mainfrom
QArtur99:master
Open

Fix eternal loading progress #719 #697#738
QArtur99 wants to merge 2 commits into
nickbutcher:mainfrom
QArtur99:master

Conversation

@QArtur99

@QArtur99 QArtur99 commented Jun 22, 2019

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Fix eternal refreshing and empty state in HomeActivity
Fix eternal refreshing in Search
Feat placeholder when data are not available same like for no filters

💡 Motivation and Context

When only "Product Hunt" filter was selected and no data was available from the response, the progress was showing infinitely, also function onLoadMore in InfiniteScrollListener was refreshing infinitely when data was not available.
#719

if searched data was not available, LiveData didn't trigger because data loading function didn't assign an empty list to searchResult
#697

💚 How did you test it?

Manually

📝 Checklist

  • I ran ./gradlew spotlessApply before submitting the PR
  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing

🔮 Next steps

📸 Screenshots / GIFs

719

@QArtur99 QArtur99 changed the title Fix eternal loading progress #719 Fix eternal loading progress #719 #697 Jun 23, 2019

@nickbutcher nickbutcher left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix! Can we change the text & CTA shown if there is an error to:

"Error loading data" + a "Retry" button which kicks off another load (rather than directing them to the filters).

@keyboardsurfer

Copy link
Copy Markdown
Collaborator

Hi @QArtur99, can you confirm that you're adressing the requested changes?

@QArtur99

QArtur99 commented Dec 30, 2019

Copy link
Copy Markdown
Contributor Author

Hi @QArtur99, can you confirm that you're adressing the requested changes?

Hi, @keyboardsurfer I opened new pr cuz I couldn't connect to this pr. New pr: #814

@QArtur99

QArtur99 commented Dec 30, 2019

Copy link
Copy Markdown
Contributor Author

Thanks for the fix! Can we change the text & CTA shown if there is an error to:

"Error loading data" + a "Retry" button which kicks off another load (rather than directing them to the filters).

Hey, @nickbutcher I changed text & CTA, and I opened new pr #814

@QArtur99
QArtur99 requested a review from nickbutcher February 7, 2020 20:35
@codingjeremy
codingjeremy changed the base branch from master to main September 29, 2020 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants