Skip to content

Handle refresh failure#956

Merged
zaelgohary merged 5 commits into
developmentfrom
development_refactor_news
Apr 3, 2025
Merged

Handle refresh failure#956
zaelgohary merged 5 commits into
developmentfrom
development_refactor_news

Conversation

@zaelgohary
Copy link
Copy Markdown
Contributor

@zaelgohary zaelgohary commented Mar 17, 2025

Changes

  • Refactor news, use infinite_scroll_pagination
  • Add Try again button in News, Wallets, DAO, Farm, Council
  • Remove ScrollController
  • Change articlesPerPage to 20 instead of 5
  • Remove redundant code

Related Issues

Tested Scenarios

  • Navigate to news
  • Reload
  • Scroll to reload more articles
  • Disable wifi before refreshing the page to see try again button

@zaelgohary zaelgohary changed the title Refactor News Screen Add try again btn Mar 17, 2025
@zaelgohary zaelgohary changed the title Add try again btn Handle refresh failure Mar 17, 2025
@zaelgohary zaelgohary marked this pull request as draft March 18, 2025 16:38
@zaelgohary zaelgohary marked this pull request as ready for review April 2, 2025 08:31
@zaelgohary zaelgohary merged commit a3149c0 into development Apr 3, 2025
@zaelgohary zaelgohary deleted the development_refactor_news branch April 3, 2025 01:30
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.

2 participants