Skip to content
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

Various fixes #276

Merged
12 commits merged into from
Aug 25, 2021
Merged

Various fixes #276

12 commits merged into from
Aug 25, 2021

Conversation

monoidic
Copy link
Collaborator

@monoidic monoidic commented Aug 23, 2021

This PR:

@ghost ghost requested a review from e3rd August 23, 2021 13:44
@ghost ghost added this to the 3.0.1 milestone Aug 23, 2021
@ghost
Copy link

ghost commented Aug 23, 2021

Wow, thanks again!

Copy link
Member

@e3rd e3rd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good!

@monoidic
Copy link
Collaborator Author

Added some variable declarations that were missing, and fixed an UI/UX issue where clicking on the glyph inside of the start/stop/reload bot buttons would not work properly.

@ghost
Copy link

ghost commented Aug 24, 2021

Are there more fixes coming? :)

As soon as you deem it ready, we can release version 3.0.1.

@monoidic
Copy link
Collaborator Author

I think that should be it for now.

@monoidic monoidic marked this pull request as draft August 25, 2021 10:07
@monoidic monoidic marked this pull request as ready for review August 25, 2021 10:17
@monoidic
Copy link
Collaborator Author

... :P

@ghost ghost merged commit bb0d6a9 into certtools:develop Aug 25, 2021
@ghost
Copy link

ghost commented Aug 25, 2021

thanks! also merged into maintenance for the 3.0.1 release

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants