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

Instance information #262

Merged
merged 6 commits into from
Nov 23, 2017
Merged

Instance information #262

merged 6 commits into from
Nov 23, 2017

Conversation

dethos
Copy link
Collaborator

@dethos dethos commented Nov 10, 2017

This pull request is the minimal implementation of the issue #260, that is in discussion.

The idea is to give user a little more context about who is running a particular "instance" or "deployment" of Hawkpost.

@dethos dethos mentioned this pull request Nov 17, 2017
@dethos dethos requested a review from pjaneiro November 17, 2017 18:30
"\">Open an issue on the project board with your question</a> "
msgstr ""
"<a href=\"https://github.com/whitesmith/hawkpost/issues\" class=\"testelinks"
"\">Abra uma questão no forúm problemas do projecto</a> "
Copy link
Collaborator

Choose a reason for hiding this comment

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

@dethos Typo on "forúm" (should be "fórum").
Additionally, the phrase doesn't seem correct, maybe it was re-written and lost some of its original form?

.env.sample Outdated
SENTRY_URL=

# (Optional) Information about the hawkpost instance and administrator
SUPPORT_NAME=
Copy link
Collaborator

Choose a reason for hiding this comment

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

@dethos Since SUPPORT_NAME and SUPPORT_EMAIL are rendered unconditionally (i.e. there's no if testing if they have been defined) we should probably mark them as mandatory; while omitting them does not break functionality it causes the labels to appear next to "nothing", and the goal of this PR is to distinguish a custom instance from https://hawkpost.co, so I see no harm in advising the user to provide them anyway.

SUPPORT_NAME = os.environ.get("SUPPORT_NAME", "")
SUPPORT_EMAIL = os.environ.get("SUPPORT_EMAIL", "")
INSTANCE_DESCRIPTION = os.environ.get("INSTANCE_DESCRIPTION", "")
VERSION = "1.1.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

@dethos This is going to be the canonical source for the Hawkpost's version? Shouldn't this be moved way up this file, since it should be updated frequently? What is going to be our version bump policy from now on?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This should be in sync with the release on the project repository. But I'm open to discuss a better approach to this, so feel free to open an issue. 👍

Copy link
Collaborator

@jlbribeiro jlbribeiro left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jlbribeiro jlbribeiro merged commit 68b1e66 into master Nov 23, 2017
@jlbribeiro jlbribeiro deleted the instance-information branch November 23, 2017 19:52
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.

2 participants