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

Part of the text is referenced as xml, and part of the Chinese text is added #41

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aacai
Copy link

@aacai aacai commented Feb 3, 2025

Part of the text is referenced as xml, and part of the Chinese text is added.

@shubham0204 shubham0204 self-requested a review February 3, 2025 01:43
@@ -2,7 +2,7 @@
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:tools="http://schemas.android.com/tools">

<uses-permission android:name="android.permission.INTERNET"/>
<uses-permission android:name="android.permission.INTERNET" />
Copy link
Owner

Choose a reason for hiding this comment

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

Can we remove all changes from this file?
Changes apart from the Chinese translation should be removed, including formatting and linting changes.

@@ -4,15 +4,15 @@ plugins {
kotlin("plugin.serialization") version "2.1.0"
}

val ktorVersion = "3.0.2"
val ktorVersion = "3.0.3"
Copy link
Owner

Choose a reason for hiding this comment

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

Please revert all changes from this file.

@@ -1,6 +1,6 @@
#Wed Nov 06 17:43:42 IST 2024
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-8.9-bin.zip
distributionUrl=https\://mirrors.tencent.com/gradle/gradle-8.9-bin.zip
Copy link
Owner

Choose a reason for hiding this comment

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

The Gradle version shouldn't be a part of this PR. Please remove all changes from this file.

android.nonTransitiveRClass=true
android.inject.testOnly=false
Copy link
Owner

Choose a reason for hiding this comment

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

Please remove all changes from this file.

.padding(16.dp)
.padding(paddingValues)
.verticalScroll(rememberScrollState()),
Modifier
Copy link
Owner

Choose a reason for hiding this comment

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

Please remove all changes due to formatting. FYI, I use ktlint for formatting and linting in the project.

@shubham0204
Copy link
Owner

A Chinese translation would be a great addition to the app! Also, I would request you to write the commit message (try keeping only 1 commit in this PR) in English as all other commit messages are also in English.

@aacai
Copy link
Author

aacai commented Feb 3, 2025

Thank you for your correction. It is also my first time to do pr. I will regulate my behavior.

@aacai
Copy link
Author

aacai commented Feb 3, 2025

Please reject this pr, I will re-standardize a pr submission

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