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
feat-mobile: setup activity flow #5239
feat-mobile: setup activity flow #5239
Conversation
…le/setup-activity-details
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work 🚀
function handleOnClick(): void { | ||
$selectedActivity = activity | ||
$activityRouter?.goTo(ActivityRoute.Details) | ||
onClick() | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To make the component reusable in the future possibily with desktop, I wouldnt handle this logic here but let the view be the one to decide, so ideally we should add a onActivityClick
to packages/mobile/components/ActivityList.svelte
and handle the logic directlty in ActivityList
, I think it might be better this way
onClick() | ||
} | ||
|
||
function onReject(): void { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here, routing logic shouldnt be in a component but on the routing views
$activityRouter?.goTo(ActivityRoute.Reject) | ||
} | ||
|
||
async function onClaim(): Promise<void> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here, I would move all this out
…le/setup-activity-details
Summary
This PR initializes the activity flow for stardust mobile
Changelog
Relevant Issues
Testing
Enable
dashboard.activity.details
anddashboard.activity.actions
inpackages/mobile/features/features.ts
You can use branch
dev-mode/mobile-stardust
for testing purposes.From the root of the repo in this PR's branch
refactor/mobile-routers-after-refactor
you can do:Platforms
Instructions
...
Checklist