mirror of
https://github.com/UpsilonNumworks/Upsilon.git
synced 2026-05-09 08:25:44 +02:00
[PR #134] [MERGED] [apps/settings] Fix assert in about submenu when username enabled #278
Labels
No labels
bug
duplicate
easy
enhancement
enhancement
fixed
fixed
good first issue
hard
invalid
pull-request
wontfix
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/Upsilon#278
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/UpsilonNumworks/Upsilon/pull/134
Author: @Yaya-Cout
Created: 1/25/2022
Status: ✅ Merged
Merged: 1/25/2022
Merged by: @Lauryy06
Base:
upsilon-dev← Head:upsilon-dev-about-assert-fix📝 Commits (3)
071f433[apps/settings] Fix assert in about submenu when username enabledd47282bRemove some broken thinks5243682Fix number of cells📊 Changes
2 files changed (+1 additions, -2 deletions)
View changed files
📝
apps/settings/sub_menu/about_controller.cpp(+0 -1)📝
apps/settings/sub_menu/about_controller.h(+1 -1)📄 Description
When we go in end of about submenu of settings in debug mode, the simulator crash because of the total number of cells, this pull request fixes that and fix also the same bug with username enabled (there are 2 cells more than
k_totalNumberOfCell)🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.