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

(hotfix to develop) misc bugs #163

Merged
merged 2 commits into from
Apr 23, 2021
Merged

(hotfix to develop) misc bugs #163

merged 2 commits into from
Apr 23, 2021

Conversation

gweiying
Copy link
Contributor

@gweiying gweiying commented Apr 23, 2021

This PR changes the library used for encoding on the backend, standardising it on the Base64 library. The other library used, base-64 does not support encoding foreign language, hence users were experiencing difficulty saving resource files containing foreign language characters, as they were being encoded with base-64.

Error message on backend logs:
Screenshot 2021-04-22 at 6 13 32 PM

Separately, this PR also fixes a bug where page content is not added to files moved to/ out of subfolders, which causes pages to appear blank after moving out of/ to a subfolder.

@gweiying gweiying changed the title Fix/misc bugs (hotfix) misc bugs Apr 23, 2021
@gweiying gweiying changed the title (hotfix) misc bugs hotfix to develop: misc bugs Apr 23, 2021
@gweiying gweiying changed the title hotfix to develop: misc bugs (hotfix to develop) misc bugs Apr 23, 2021
@gweiying gweiying requested a review from alexanderleegs April 23, 2021 06:21
Copy link
Contributor

@alexanderleegs alexanderleegs left a comment

Choose a reason for hiding this comment

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

lgtm, good catch

@gweiying gweiying merged commit babd599 into develop Apr 23, 2021
@gweiying gweiying deleted the fix/misc-bugs branch April 23, 2021 07:10
alexanderleegs pushed a commit that referenced this pull request Apr 29, 2021
* fix: change base64 encoding

* fix: missing content during page move
harishv7 pushed a commit that referenced this pull request Feb 17, 2023
* fix: change base64 encoding

* fix: missing content during page move
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