-
Notifications
You must be signed in to change notification settings - Fork 68
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
[fix] Fixed render issue in popup page #113 #114
[fix] Fixed render issue in popup page #113 #114
Conversation
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.
Tested, works, solution looks good, not sure if we want to start with Integration testing 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.
Wait, noticed a problem, when I try to create a new CA from from certificate's add form, I don't get the option to select if I want create new or import.
This change should only affect the change page! 😄
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.
Wait, noticed a problem, when I try to create a new CA from from certificate's add form, I don't get the option to select if I want create new or import.
This change should only affect the change page! smile
Definitely.
@niteshsinha17 please spend time testing that your changes do not affect negatively the existing features.
@atb00ker Thanks for spotting this. I have fixed it. Please check. |
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.
Thanks @niteshsinha17 and @atb00ker for your precious review & testing 👍
@nemesisdesign Before writing tests I wanted to show the implementation. Is it ok? I will need to use selenium in the test.
closes #113