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

Change the report url to github new issue link #473

Closed
wants to merge 2 commits into from

Conversation

eskimo
Copy link

@eskimo eskimo commented Feb 20, 2022

Since the previous link doesn't work, I think the github new issue link is a good replacement.

@rithvikvibhu
Copy link
Collaborator

Thanks @eskimo for the PR!

closes #143, closes #180, LGTM.

@Falci
Copy link
Contributor

Falci commented Mar 11, 2022

Didn't see your PR before.

We could define the links in a single place and reference them wherever we need.
My first idea was to define them in the package.json.

WDYT?

@eskimo
Copy link
Author

eskimo commented Mar 14, 2022

I know nothing about this project at all, so changing the links how they were originally defined is basically the best I can do :P

@rithvikvibhu
Copy link
Collaborator

#481 replaces this and uses package.json values. Thanks again for this PR @eskimo!

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.

[Feature] App Version Popup, Cleanup Windows: help menu is electron stuff
3 participants