-
Notifications
You must be signed in to change notification settings - Fork 5
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
Layout for "Manage starnames/iovnames/names" views #1048
Layout for "Manage starnames/iovnames/names" views #1048
Conversation
991be4e
to
ab0ce3b
Compare
import AccountManage from "./manage"; | ||
import AccountRegister from "./register"; | ||
import AccountUpdate from "./update"; | ||
|
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.
Maybe move AccountEntity
type from "./register"
here?
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.
Sorry, do not understand you.
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.
You have the subdirectories: manage, register, and update. All of them use the AccountEntity
type and AccountProps
interface from the register subdirectory. Since the three of them use them, it no longer makes sense to have those declared in register, they should be declared on the root, which is this file.
So between the imports and the export you can declare:
type AccountEntity = "iovname" | "starname" | "name";
interface AccountProps {
entity: AccountEntity;
}
Add them to the export, and removed them from the register's index.tsx.
Closes #1021.
Starname manage
Starname update
Name manage
Name update
Iovname manage
Iovname update