Skip to content

Fix broken build#378

Merged
Rokt33r merged 3 commits intoBoostIO:masterfrom
sbencoding:master
Mar 26, 2020
Merged

Fix broken build#378
Rokt33r merged 3 commits intoBoostIO:masterfrom
sbencoding:master

Conversation

@sbencoding
Copy link
Copy Markdown
Contributor

Hi!
I tried to build the project but ran into several problems.
In this PR I post the changes I've made to be able to build the project.

  1. The Spanish locale file was renamed from esEs.ts to esES.ts because src/lib/i18n.ts referenced the file with the big S at the end.
  2. The build doesn't run without a .env file in the project root. So contributors would need to create an empty one for the build to run. I'm not really familiar with the .env system used here, so I guess you could provide a better documentation for this problem than I did. My understanding is that you use that file for storing tokens for analytics but I didn't investigate it thoroughly.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 23, 2020

CLA assistant check
All committers have signed the CLA.

@sbencoding
Copy link
Copy Markdown
Contributor Author

Filename issue fixed in f4012ec
I still think we should add a reference in the readme file for the .env file, because the build doesn't run without it

@Rokt33r
Copy link
Copy Markdown
Member

Rokt33r commented Mar 26, 2020

@sbencoding Yeah, I'm going to merge this soon.

@Rokt33r Rokt33r merged commit e0a1dd8 into BoostIO:master Mar 26, 2020
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.

3 participants