Skip to content

Conversation

@jhdcruz
Copy link
Contributor

@jhdcruz jhdcruz commented Apr 27, 2020

Description

Changes Remove to Delete Storage.

Other localization languages needs update due to this change.

Copy link
Contributor

@marilari88 marilari88 left a comment

Choose a reason for hiding this comment

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

There are still many labels and description hardcoded. As said in #433 we've is a partally traslated application. I think we should start listing all the missing traslations and then work to introduce them in the code.

@ZeroX-DG ZeroX-DG added the awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. label Apr 29, 2020
@jhdcruz jhdcruz requested a review from ZeroX-DG April 29, 2020 09:50
@jhdcruz
Copy link
Contributor Author

jhdcruz commented Apr 29, 2020

The i18n support for translation is done. However, other languages might need to be re-checked/proofread since I just inserted the key storage.name after the strings.

Copy link
Member

@ZeroX-DG ZeroX-DG left a comment

Choose a reason for hiding this comment

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

That's cool, I'll open an issue for everyone to fix their language later 👍

@ZeroX-DG ZeroX-DG added approved 👍 Pull request has been approved by sufficient reviewers. and removed awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. labels Apr 30, 2020
@Rokt33r Rokt33r added this to the v0.5.0 milestone May 1, 2020
@Rokt33r Rokt33r merged commit d642196 into BoostIO:master May 1, 2020
@jhdcruz jhdcruz deleted the update-storage-strings branch May 1, 2020 07:20
Rokt33r pushed a commit that referenced this pull request May 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved 👍 Pull request has been approved by sufficient reviewers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants