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

Enhance code structure and book management #142

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

YakeDev
Copy link
Contributor

@YakeDev YakeDev commented Oct 25, 2024

  • Refactored the code into ES6 modules, allowing for better organization and reusability of functionalities.
  • Most features are now encapsulated in modules, making maintenance and testing easier.
  • Integrated a books.json file to dynamically add all books.
  • Created the reading.html page to display book metadata dynamically.
  • Developed the reading.js file to manage the logic for the reading.html page.

Note: the chapter progressbar doesn't work, need help because I didn't find were the bug comes

github-actions bot and others added 3 commits October 25, 2024 22:49
Refactored the code into ES6 modules, allowing for better organization and reusability of functionalities.
Most features are now encapsulated in modules, making maintenance and testing easier.
Integrated a books.json file to dynamically add all books.
Created the reading.html page to display book metadata dynamically.
Developed the reading.js file to manage the logic for the reading.html page.
Copy link

netlify bot commented Oct 25, 2024

Deploy Preview for lisbook ready!

Name Link
🔨 Latest commit 73b8b61
🔍 Latest deploy log https://app.netlify.com/sites/lisbook/deploys/671c2279f68fed0008adba4c
😎 Deploy Preview https://deploy-preview-142--lisbook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@YakeDev
Copy link
Contributor Author

YakeDev commented Oct 25, 2024

Hey @Ctoic, can you check now? pls

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.

1 participant