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

[wip] feat: add monitor overflow to develop branch #958

Draft
wants to merge 28 commits into
base: develop
Choose a base branch
from

Conversation

alexbarnsley
Copy link
Member

@alexbarnsley alexbarnsley commented Sep 12, 2024

Summary

https://app.clickup.com/t/86du0hhmn

todo:

  • check implementation
  • implement livewire 3 changes to handle issues with favourite sorting
  • updates to tests - need to cater for the differences in database schema
  • Copy across new Overflow tests to handle coverage - (on hold - see Slack Thread)

Checklist

  • I checked my UI changes against the design and there are no notable differences
  • I checked my UI changes for any responsiveness issues
  • I checked my UI changes in light AND dark mode
  • I checked my (code) changes for obvious issues, debug statements and commented code
  • I opened a corresponding card on Clickup for any remaining TODOs in my code
  • I added a short description on how to test this PR (if necessary)
  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

alexbarnsley and others added 28 commits September 12, 2024 23:38
…com:ArdentHQ/arkscan into feat/add-monitor-overflow-to-develop-branch
…com:ArdentHQ/arkscan into feat/add-monitor-overflow-to-develop-branch
…com:ArdentHQ/arkscan into feat/add-monitor-overflow-to-develop-branch
…com:ArdentHQ/arkscan into feat/add-monitor-overflow-to-develop-branch
…com:ArdentHQ/arkscan into feat/add-monitor-overflow-to-develop-branch
Base automatically changed from feat/update-laravel to develop October 23, 2024 13:20
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.

1 participant