Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

first pr #6

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

first pr #6

wants to merge 4 commits into from

Conversation

yurtemre7
Copy link
Contributor

I've tried scanning your code and try to understand that and while I looked at specific points I've found some corrections or way to to improve it.
The only critical thing changed here is the description of an anime, because some times there where some "special chars" like "hyphens (-)" that were not displayed correctly.
The other changes occurred while auto-formatting the code by my IDE, (we better have the same format style though the next time)

@yurtemre7
Copy link
Contributor Author

is this gonna get merged or xd?

@simrat39
Copy link
Owner

@yurtemre7 Yes sorry I'll fixup tv first then look at iOS

Copy link

@rafifos rafifos left a comment

Choose a reason for hiding this comment

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

Sorry, but:
https://http.cat/406

@@ -20,6 +20,9 @@
# is commented out by default.
#.vscode/

# iOS
ios
Copy link

Choose a reason for hiding this comment

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

Sorry I have OCD.

Suggested change
ios
ios/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

XD all fine

(currentAmount?.toString() ?? "?") +
" / " +
(totalAmount?.toString() ?? "?"),
(currentAmount?.toString() ?? "?") + " / " + (totalAmount?.toString() ?? "?"),
Copy link

Choose a reason for hiding this comment

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

I have approximately 0 confidence with Dart but it supports interpolation?

Copy link
Owner

Choose a reason for hiding this comment

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

yes dart supports string interpolation

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants