-
Notifications
You must be signed in to change notification settings - Fork 19
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
Home dashboard UI fix #69
Conversation
Minor issue but what should we do with the scroll bar? @QUDBS |
src/app/screens/home/index.tsx
Outdated
} = useSelector((state: StoreState) => state.walletState); | ||
|
||
const fetchFeeMultiplierData = async () => { | ||
const response: FeesMultipliers = await fetchAppInfo(); | ||
dispatch(FetchFeeMultiplierAction(response)); | ||
}; | ||
|
||
const getActiveAccountList = async ( |
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.
Should this logic be in the core library since it's used in both the app and the extension. It's also quite isolated.
@@ -8,6 +8,8 @@ import { useNavigate } from 'react-router-dom'; | |||
import { Ring } from 'react-spinners-css'; | |||
import useWalletReducer from '@hooks/useWalletReducer'; | |||
|
|||
declare const VERSION: string; |
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.
how is this version string set?
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.
new webpack.DefinePlugin({ |
in the webpack.config.js the version variable is set and can be accessed anywhere in the app to avoid hardcoding the extension version number
src/app/utils/helper.ts
Outdated
@@ -75,3 +76,22 @@ export async function isValidURL(str: string): Promise<boolean> { | |||
} | |||
return false; | |||
} | |||
|
|||
export async function checkAccountActivity( |
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.
This should probably also be in the core library.
…-extension into imamahzafar/home-dashboard-ui-fix
* fix: spacing issues * added scrollbar in rare sat detail * fix: scrollbar in confirm screen * fix: scrollbar in rarity screen * fix: send errors * removed console log * fix: rare sat bundle navigation * fix: rare detail text * fix: confirm UI * fix: rare sats bundle navigation
PR Type
What kind of change does this PR introduce?
Fixes ui bugs found in the main dashboard screen
loads all active accounts when user logs in
fixes bug in account screen