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

Allow option for hiding version information #25

Merged
merged 2 commits into from
Jun 27, 2018

Conversation

alexanderturinske
Copy link
Contributor

@alexanderturinske alexanderturinske commented May 31, 2018

@rhysd
Copy link
Owner

rhysd commented Jun 12, 2018

I'm sorry that I failed to catch this PR. Let me review.

package.json Outdated
@@ -1,6 +1,6 @@
{
"name": "about-window",
"version": "1.11.0",
"version": "1.12.0",
Copy link
Owner

Choose a reason for hiding this comment

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

Could you revert this change? Let me create another commit to bump version to create a tag commit after merging this PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will do!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@rhysd
Copy link
Owner

rhysd commented Jun 12, 2018

@alexanderturinske Could you show me the screenshot how it looks like when setting use_version_info to false?
And could you please add the option to .d.ts file and README.md?

- add new option to README
- revert version change
@alexanderturinske
Copy link
Contributor Author

@rhysd

  • Screenshot added to the commit.
  • Added option to README
  • Option was already in .d.ts, but not documented in README that it was

Copy link
Owner

@rhysd rhysd left a comment

Choose a reason for hiding this comment

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

Looks good. Thank you for the modifications.

@rhysd
Copy link
Owner

rhysd commented Jun 27, 2018

I will publish new version after merging #26

@rhysd rhysd merged commit 310b7f6 into rhysd:master Jun 27, 2018
@bampakoa
Copy link
Contributor

bampakoa commented Jul 5, 2018

@rhysd I think that this PR did not make it to the latest release 🤔 It seems that the if statement is missing from renderer.js

@rhysd
Copy link
Owner

rhysd commented Jul 6, 2018

@bampakoa Thank you for reporting that. I released 1.12.1. Could you try it?

@bampakoa
Copy link
Contributor

bampakoa commented Jul 6, 2018

@rhysd it works fine in 1.12.1 Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants