-
Notifications
You must be signed in to change notification settings - Fork 6
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
[#21] display version in admin #112
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.
Similar situation as mentioned in this comment: open-zaak/open-notificaties#155 (comment)
Please rebase on master, quick-start action is fixed there |
e7ba886
to
6a8cf5c
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.
@@ -36,6 +36,12 @@ DO NOT PUT ANY TARGET APP-SPECIFIC RULES HERE. | |||
|
|||
--object-tools-bg: #888; | |||
--object-tools-hover-bg: #{$color_dark}; | |||
|
|||
// version styling | |||
--version-fg: #{$color-secondary-dark}; |
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.
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.
c006fd4
to
a6f0325
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.
Approving, but let's wait until the PR in OAF is merged and released and then pin it to the pypi release here
f6522db
to
5ca35b6
Compare
Partially fixes maykinmedia/open-api-framework#21 :