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

Ebooks: fix scanning and crafting menu info #79354

Merged
merged 4 commits into from
Jan 28, 2025

Conversation

o175
Copy link
Contributor

@o175 o175 commented Jan 26, 2025

Fix "Written in: ebook" crafting menu; Scanning should be allowed with backlight off.

fixes #79301, fixes #79317

Summary

Bugfixes "Fix written in: ebook; Allow scanning books with backlight off"

Purpose of change

There should be an option scan books with backlight off. It does not matter if the backlight on or off for taking photos on e-ink tablet. Because it's e-ink.
Written in: ebook should be present in crafting menu.

Describe the solution

Add EBOOKSAVE flag to e-ink tablet; Add ebooks to inventory::get_binned_items()

Describe alternatives you've considered

Testing

Manual testing: find non-memorized recipe in crafting menu. Scan e-book with screen off.

Additional context

#79317 is fixed. Not sure if it's by this PR or earlier.

Crafting menu lacks "Written in:" info; Scanning should be allowed with
backlight off.

fixes CleverRaven#79301, fixes CleverRaven#79317
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) json-styled JSON lint passed, label assigned by github actions labels Jan 26, 2025
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Jan 26, 2025
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Jan 26, 2025
@Maleclypse Maleclypse merged commit bf468b8 into CleverRaven:master Jan 28, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can Not Train "computer (advanced)" Due to "HackPRO" Not Being Found Ebooks are broken
2 participants