-
Notifications
You must be signed in to change notification settings - Fork 1
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(cip-1694): restyle intro page #39
Conversation
vetalcore
commented
Jul 19, 2023
•
edited
Loading
edited
ui/cip-1694/src/App.tsx
Outdated
<Footer /> | ||
<Toaster | ||
toastOptions={{ | ||
className: '', |
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.
of course use the class, i was in rush. and colours from theme. thanks.
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.
resolved
@@ -1,4 +1,3 @@ | |||
export const USER_SESSION_KEY = 'userInSession'; | |||
// TODO: do we nned to think about localizations? |
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.
yes we need to add i18n
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.
will add in a separate pr if approved
const time = new Date(); | ||
const endTime = moment('09-01-2023', 'MM-DD-YYYY'); // summit date | ||
time.setSeconds(time.getSeconds() + endTime.diff(time, 'seconds')); // time left for summit date | ||
const endTime = moment('09-01-2023', 'MM-DD-YYYY').format('D MMMM YYYY, h:mm a'); // summit date |
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.
can we add this as env variable?
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.
resolved
})); | ||
|
||
// TODO: could we rename this to be more a generic one? just "slides"? |
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.
then it goes into components/common
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.
resolved
ui/cip-1694/src/pages/Vote/Vote.tsx
Outdated
@@ -116,7 +116,7 @@ const Vote = () => { | |||
votePower: votingPower, | |||
}); | |||
signMessage(canonicalVoteInput, async (signature, key) => { | |||
// TODO: castAVoteWithDigitalSignature are incompatible with requestVoteObject | |||
// FIXME: castAVoteWithDigitalSignature are incompatible with requestVoteObject |
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.
what do you mean ? :)
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.
nvm, already resolved
Great work. looks awesome |
Just a note i am off for 2 weeks, this PR is good to merge just take care of failed e2e tests. Thanks |
9522cd3
to
529c97e
Compare
559364b
to
2f58f55
Compare
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.
LGTM
2f58f55
to
0f66c2c
Compare
6978744
to
2e696e3
Compare