Skip to content

Conversation

@jakobh7
Copy link

@jakobh7 jakobh7 commented Nov 15, 2018

This code has all translatable words separated out for ease of translation. Let me know if I've missed anything or I need to add anything

language_name: language_name
});
}

Copy link
Author

Choose a reason for hiding this comment

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

These methods will be updated over the weekend to actually implement reading from PO files

Copy link
Contributor

Choose a reason for hiding this comment

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

@jakobh7 Nice, would the PO files stored at the frontend or have to make a request from RESTful API?
kobh7 Do we need to think about what default values for those strings are? Could be English for example or leave it blank?

Copy link
Author

Choose a reason for hiding this comment

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

I think the PO files on the front end shouldn't have to make requests to the RESTful API. I was thinking the default values for the strings would be English, but that's just because that's the common language we're writing in currently.

Copy link
Contributor

@pannapat pannapat left a comment

Choose a reason for hiding this comment

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

I just added a question related to the service part.

language_name: language_name
});
}

Copy link
Contributor

Choose a reason for hiding this comment

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

@jakobh7 Nice, would the PO files stored at the frontend or have to make a request from RESTful API?
kobh7 Do we need to think about what default values for those strings are? Could be English for example or leave it blank?

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