Skip to content

home-environment: fix evaluation error on undefined lang.base#3

Closed
dermetfan wants to merge 1 commit intonix-community:masterfrom
dermetfan:fix-lang-base
Closed

home-environment: fix evaluation error on undefined lang.base#3
dermetfan wants to merge 1 commit intonix-community:masterfrom
dermetfan:fix-lang-base

Conversation

@dermetfan
Copy link
Copy Markdown
Collaborator

Triggered by home.sessionVariableSetter = "pam";.

@rycee
Copy link
Copy Markdown
Member

rycee commented Feb 12, 2017

Many thanks again! I rebased the PR into master in fa73a7f.

I think my original idea was that it wouldn't make much sense to set language options without having a base language. But if you want to maintain the system language and only change, e.g., the monetary language setting then of course the base has to be unset here. Good stuff :-)

@rycee rycee closed this Feb 12, 2017
@rycee
Copy link
Copy Markdown
Member

rycee commented Feb 12, 2017

By the way, I sent you a collaboration invite that you can accept if you'd like to be able to do direct commits. If you accept please keep the commit message format that has been used up to now and if you make non-trivial changes (I would consider this a trivial change) then please open a PR.

@dermetfan dermetfan deleted the fix-lang-base branch February 12, 2017 14:00
@dermetfan
Copy link
Copy Markdown
Collaborator Author

Thank you for the trust! I'll try my best when I come across an issue again.

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