-
Notifications
You must be signed in to change notification settings - Fork 21
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: add organizer dashboard #276
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
3fe14f4
to
8eb8abc
Compare
8eb8abc
to
3f25d13
Compare
23eefef
to
f982c75
Compare
3f25d13
to
2533c43
Compare
2533c43
to
dee1074
Compare
dee1074
to
4d7f321
Compare
4d7f321
to
0a76b5d
Compare
0a76b5d
to
703fc8c
Compare
703fc8c
to
93c3ca8
Compare
93c3ca8
to
09c1a2c
Compare
09c1a2c
to
9ee35e3
Compare
9ee35e3
to
ac4d137
Compare
748a847
to
747983b
Compare
747983b
to
91df7ae
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.
@kittybest thanks, there are some comments and why PR is for dev
branch?
maci-platform repo has main
branch as current changes and tags
for stable versions.
packages/interface/src/features/admin/components/DeployContractsButtons.tsx
Outdated
Show resolved
Hide resolved
onSubmit={(round) => { | ||
create.mutate(round); | ||
}} |
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.
Don't use arrow functions inside jsx
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 should we also change the one inside features/applications/components/ApplicationForm.tsx
?
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.
If you see it's used somewhere, it's better to change it because it affects performance.
packages/interface/src/features/admin/components/DeployRoundsButtons.tsx
Outdated
Show resolved
Hide resolved
24c3ede
to
c70854f
Compare
91df7ae
to
019a61b
Compare
maybe we should find a better name for it? or are you saying why merge this to since currently the only the |
019a61b
to
2e5ff7e
Compare
I think let's keep dev for now as we discussed before. When we get rid of mock data and integrate new logic with the ui, we can merge dev into main. |
2e5ff7e
to
82de1d4
Compare
5c4c908
to
7c509d5
Compare
Update
Keep this as a separate branch (not merging this), wanna integrate registry stuff on the
dev
branch and merge it tomain
first.Description
/admin
page for organizer dashboardRoundContext
for managing the organizer dashboard global UI state