Skip to content

[Imported] Feat/noatcards support #404

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 147 commits into
base: main
Choose a base branch
from
Open

[Imported] Feat/noatcards support #404

wants to merge 147 commits into from

Conversation

ido777
Copy link
Owner

@ido777 ido777 commented Apr 6, 2025

Imported from donnemartin/system-design-primer#519

Original author: @kevintruong

noat.cards is simple SR learning system work base on telegram chat bot. By native support markdown instead of a special file format it will help us to easy to reuse/develop document which is easy to convert to note card for learning

  • Support native markdown SRS noat.cards

donnemartin and others added 30 commits December 3, 2017 07:09
@donnemartin
Update Japanese translation link (#118)
c3f080f
@koudaiii @donnemartin
ja: Fix typo (#119)
a4973a0
@donnemartin
Make some minor wording/formatting changes (#120)
748944c
@fluency03
add Twitter Handles 3,000 Images Per Second
dd124c4
@fluency03 @donnemartin
Add Netflix in Company Architectures (#122)
11b9d72
@donnemartin
Add Vietnamese translation link (#128)
8702199
@fluency03 @donnemartin
Add Scaling Uber in Company Architecture (#123)
156a67e
@donnemartin
Update master-slave section anchor (#129)
ebcec73
@fluency03 @donnemartin
Add Facebook Live Streams (#125)
0263906
@donnemartin
Update contributing guidelines - PR squash (#135)
12946d3
@hakaari @donnemartin
Update re:Invent url (#137)
61bc6f7
@fluency03
Merge branch 'master' into add-twitter
e86ed77
@kevingo @donnemartin
Add Traditional Chinese translation (#133)
9712a12
@kevingo @donnemartin
zh-TW: Add proper line break to links (#139)
82868d3
@donnemartin
Add Greek translation link (#140)
7272a9b
@donnemartin
Convert all .py files to be valid Python (#98)
de3c13c
@robertding @donnemartin
zh-Hans: Fix typos (#141)
94c158e
@donnemartin
Add Spanish translation link (#142)
7c38532
@donnemartin
Add missing colons to class methods (#143)
3079562
@donnemartin
Add missing self variables to Deck of Cards solution (#145)
d90b562
@donnemartin
Add deque import to Social Graph solution (#147)
7e316ce
@donnemartin
Change LARGE to VehicleSize.LARGE in parking lot solution (#146)
f552598
@htkuan @donnemartin
zh-TW: Fix comment format (#155)
f038dbb
@donnemartin
Fix coding errors (#149)
d7d0e9d
@hulikau @donnemartin
Fix dict KeyError (#152)
f05373a
@hulikau @donnemartin
Fix dict KeyError (#153)
7a4dbb1
@donnemartin
Adding missing self variable (#158)
4a0d7fb
@donnemartin
Add missing enum imports (#157)
02a20c6
@donnemartin
Change variable seller to category in Mint solution (#159)
2368ee0
@b7wch @donnemartin
zh-Hans: Fix typo (#161)
38cb1be
ad-m and others added 23 commits July 10, 2020 21:01
@ad-m
Change disk to HDD for clarity (#295)
e9f62f1
@donnemartin
Update README.md
22054f4
@donnemartin
Resolve #281: Add design stock exchange links (#441)
c0330a0 
* Additional question, build an exchange

fixes donnemartin#281

* Update README.md

Co-authored-by: Donne Martin <donne.martin@gmail.com>
@donnemartin
Remove extraneous __init__.py (#393)
cca3e03
@danieljl
Remove Imgur dependency by storing images locally (#168)
f655458
@iravitejakumar
Fix #335: Update OSI image to Open Systems Interconnection (#447)
2fa0808
@serima
Fix translations link in PR template (#451)
a910106
@peteryao7
Update broken HBase architecture link (#481)
a078026
@sharpcodex
Fix Celery URL (#486)
cdfdeb0
@lerignoux
Fixing the indieflashblog address with archive.org since website has …
90ee959 
…been discontinued
@donnemartin
Fixing the indieflashblog address with archive.org since website has …
bce4227 
…been discontinued (#500)

@lerignoux thank you!
@trantienduchn
Fix LRU cache link (#505)
d90c40a
@kevintruong
poriting to noat.cards
3c1dd96
@kevintruong
fixed issue preview
3ef9420
@kevintruong
update more cards
badd5d7
@kevintruong
add more resource
e7319f0
@kevintruong
change to heading level two
21d619f
@kevintruong
formatting
09f0361
@kevintruong
remove wrong format symbol
50b4140
@kevintruong
add weigh to well organize
60c38b3
@kevintruong
revert to origin
3505855
@kevintruong
re-arrang follow table content
1fe9024
@kevintruong
add weigh params
62e25e7
@Copilot Copilot AI review requested due to automatic review settings April 6, 2025 20:36
@ido777
Copy link
Owner Author

ido777 commented Apr 6, 2025

Imported comment from @kevintruong

@donnemartin would you please to check my pull request, please.
Thanks

Copilot
Copilot AI reviewed Apr 6, 2025
View reviewed changes
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ido777
Copy link
Owner Author

ido777 commented Apr 6, 2025

Imported review by @an4s911

State: APPROVED

No content

@an4s911
Copy link

an4s911 commented Apr 6, 2025

Imported review by @an4s911

State: APPROVED

No content

?????

@ido777
Copy link
Owner Author

ido777 commented Apr 8, 2025

First, the context - I started working on upgrading the old repo.
The comment you asked about is copied by a small script I wrote to extract all the PR and the issues from the old repo.
This is your original approval comes from
donnemartin#519

I do want to improve it and modernize it, one of the things is to make it a place that people can practice since I think Anki is not good enough and there are many new possible solutions.
The noatcards might be a solution - I didn't look at it yet, but surely improving the practice is one of my main goals
What to help?

@ido777 ido777 force-pushed the imported-pr-519 branch from 564ad07 to 62e25e7 Compare May 14, 2025 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.