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

Improve Chinese Language Support & Fix Issue with Asset File Names #47

Merged
merged 2 commits into from
Aug 18, 2023

Conversation

Bardreamaster
Copy link
Contributor

Type of PR

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (adding a feature following a feature-request issue)
  • Improvement (improving an existing feature - includes style: and perf:commits)
  • Refactor (rewriting existing code without any feature change)
  • (!) This change is or requires a documentation update

Description

Reopen PR #44

  1. Fix asset import error caused by name in lower case.

In fronted APP, the icons be imported using capitalized filename in frontend/src/icons.ts

import Admin from '../assets/icons/AdminPanel.svg';
import Settings from '../assets/icons/Settings.svg';

But the filename is lowercase. So it causes error on fronted APP:

Unable to resolve module

So we have to unify naming to avoid import error.

1.** Fix Simplified Chinese keywords for getPosterTypeDetails .**

Changes

Assets name

Fix asset import issue by renaming the following files:

  • frontend/assets/icons/adminPanel.svg → frontend/assets/icons/AdminPanel.svg
  • frontend/assets/icons/settings.svg → frontend/assets/icons/Settings.svg

Additional Screenshots

Nope.

Additional information/context

Tested on Discourse site in Chinese. (https://forum.robook.org)

@Bardreamaster
Copy link
Contributor Author

@Kinostrome I've rebased the previous changes into alpha-v2. And I accidentally delete the original branch. So I reopen the PR to rearrange the descriptions.
And it looks the CI worked well.
Thanks for review.

Copy link
Contributor

@Kinostrome Kinostrome left a comment

Choose a reason for hiding this comment

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

This looks good, thanks again for the improvements!

@Kinostrome Kinostrome merged commit d76a552 into lexiconhq:alpha-v2 Aug 18, 2023
1 check passed
@Bardreamaster Bardreamaster deleted the fix_poster_details branch August 22, 2023 09:28
starjustice added a commit that referenced this pull request Sep 25, 2023
* sync latest version 2 changes (alpha)

* Bug fix: Fix an error with the Topic and Post image URL when using a shortUrl as the source.

* Bug fix: Post preview with a short URL uploads an image as the source.

* add tool version, fixing nodejs to version 16

* fix quickstart command

* fix react version due to expo request

* Sync latest version 2 (alpha)

* Sync latest version 2 (alpha)

* Sync latest version 2 (alpha)

* Sync latest version 2 (alpha)

* Sync latest version 2 (alpha)

* Sync latest version 2 (alpha)

* Sync latest version 2 (alpha)

* Sync latest version 2 (alpha)

* Ensure that Github Actions runs on alpha-v2 as well (#46)

Co-authored-by: Kinostrome <[email protected]>

* Improve Chinese Language Support & Fix Issue with Asset File Names (#47)

* fix assets name

* fix getting poster ditails for different language

---------

Co-authored-by: jinqi.wei <[email protected]>

* Sync latest version 2 (alpha)

* Sync latest version 2 (alpha)

* Sync latest version 2 (alpha)

* Sync latest version 2 (beta)

* Sync latest version 2

---------

Co-authored-by: Simon Biggs <[email protected]>
Co-authored-by: Gabriella Putri <[email protected]>
Co-authored-by: Kinostrome <[email protected]>
Co-authored-by: Kinostrome <[email protected]>
Co-authored-by: ShiChangshan <[email protected]>
Co-authored-by: jinqi.wei <[email protected]>
Bardreamaster added a commit to Bardreamaster/robook-forum that referenced this pull request Nov 29, 2023
…exiconhq#47)

* fix assets name

* fix getting poster ditails for different language

---------

Co-authored-by: jinqi.wei <[email protected]>
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