Skip to content

Conversation

@ignatovv
Copy link
Member

@ignatovv ignatovv commented Oct 29, 2025

Summary

  • Implemented new object settings menu design with unified interface
  • Added feature flag newObjectSettings to toggle between old and new UI
  • Integrated new menu into template editing and editor pages

@ignatovv ignatovv requested a review from a team as a code owner October 29, 2025 16:47
@claude
Copy link
Contributor

claude bot commented Oct 29, 2025

Review Summary

This PR implements a new object settings menu design with a unified interface, properly gated behind the newObjectSettings feature flag. The implementation follows good architectural patterns with clean separation of concerns.

Issues Found

ObjectSettingsMenuViewModel.swift:108

  • Typo: makeAsTempalte should be makeAsTemplate (missing 'l')

ObjectAction.swift (new menuOrder property)

  • Consider adding documentation explaining the ordering scheme

⚠️ Minor Issues - Fix typo in method name

github-actions bot and others added 2 commits October 30, 2025 10:39
Co-authored-by: Vova Ignatov <ignatovv@users.noreply.github.com>
@ignatovv ignatovv merged commit 46b4e52 into develop Oct 30, 2025
7 checks passed
@ignatovv ignatovv deleted the ios-5255-update-object-settings-menu-converted-to-project branch October 30, 2025 11:51
@github-actions github-actions bot locked and limited conversation to collaborators Oct 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants