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

feature: remove infinite scroll on activity screen #2503

Conversation

comountainclimber
Copy link
Member

What current issue(s) from Trello/Github does this address?
Kapture 2023-06-14 at 12 29 55

What problem does this PR solve?

How did you solve this problem?

How did you make sure your solution works?

Are there any special changes in the code that we should be aware of?

Is there anything else we should know?

  • Unit tests written?

app/containers/TransactionHistory/TransactionHistory.scss Outdated Show resolved Hide resolved
app/actions/transactionHistoryActions.js Outdated Show resolved Hide resolved
@comountainclimber comountainclimber merged commit e4c6143 into feature/contacts-refactor Jun 20, 2023
@comountainclimber comountainclimber deleted the feautre/remove-infinite-activity-scroll branch June 20, 2023 15:35
comountainclimber added a commit that referenced this pull request Jun 23, 2023
* contacts rewrite

* contacts progress

* Adds logic and deps for NNS

* Send panel, contact selection UX implementation

* Update history flow and code

* lint and clean up

* Delete contact actions foo

* flow fix

* flow fix

* flow fix

* lint

* Revert change and delete AddContact dir

* Remove cruft

* lint and style enhancements

* Clean up commented out styles

* clean up comments

* mock bs-neo3

* Refactor ContactFormRefactor.jsx

* feature: remove infinite scroll on activity screen (#2503)

* replace infinite scroll with pagination

* Fix tests remove logging

* Remove commented out style

* remove cruft
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