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

Fix eternal loading progress #719 #697 #738

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix eternal loading progress #719 #697 #738

wants to merge 2 commits into from

Conversation

QArtur99
Copy link
Contributor

@QArtur99 QArtur99 commented Jun 22, 2019

📢 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
Copy link
Owner

@nickbutcher nickbutcher left a comment

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
Collaborator

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

@QArtur99
Copy link
Contributor Author

QArtur99 commented Dec 30, 2019

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
Copy link
Contributor Author

QArtur99 commented Dec 30, 2019

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