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

feat: Adds nice!view functionality to repository #1

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

Conversation

Nabizzle
Copy link

@Nabizzle Nabizzle commented Oct 13, 2023

Makes the required changes to allow the use of a nice!view. Adding the nice_view_adapter and nice_view shields to the build.yaml file lead to errors during the compilation of the files. In order to address the errors, The Kconfig.defconfig file was edited to remove the ZMK_DISPLAY and LVGL sections and in the temper.dtsi file, the zephyr,display = &oled line had to be deleted. I also removed the deep sleep config and replaced it with the display config in the temper.config file.

This was tested on my own build of the Temper Keyboard before submitting the pull request.
image

Makes the required changes to allow the use of a nice!view. Adding the
nice_view_adapter and nice_view shields to the build.yaml file lead to
errors during the compilation of the files. In order to address the
errors, The Kconfig.defconfig file was edited to remove the ZMK_DISPLAY and
LVGL sections and in the temper.dtsi file, the zephyr,display = &oled
line had to be deleted. I also removed the deep sleep config and replaced it with the display config in the temper.config file
@Nabizzle
Copy link
Author

Accidentally closed, when I was trying to move my own personal changes to a different branch

@Nabizzle Nabizzle reopened this Oct 19, 2023
@raeedcho
Copy link
Owner

Thanks! I will merge this once I can verify that it works for me too.

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