Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

scene name validation on save as and rename #10347

Merged
merged 10 commits into from
Jun 18, 2024
Merged

scene name validation on save as and rename #10347

merged 10 commits into from
Jun 18, 2024

Conversation

aditya-mitra
Copy link
Collaborator

@aditya-mitra aditya-mitra commented Jun 7, 2024

Summary

user input scene name validation

Subtasks Checklist

Breaking Changes

References

https://tsu.atlassian.net/browse/IR-2453

https://tsu.atlassian.net/browse/IR-2531

https://tsu.atlassian.net/browse/IR-1446

QA Steps

image

image

Copy link
Member

@SYBIOTE SYBIOTE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, is it worth giving the valid scene name conditions its own variable so its easily reusable in the future

Copy link
Member

@HexaField HexaField left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should make the same restrictions on the backend

@aditya-mitra
Copy link
Collaborator Author

we should make the same restrictions on the backend

done and added the validation in common package

@HexaField HexaField merged commit f0b5c5a into dev Jun 18, 2024
28 checks passed
@HexaField HexaField deleted the scene-name-valid branch June 18, 2024 08:30
MbfloydIR added a commit that referenced this pull request Jun 18, 2024
* dev:
  IR-1775-Changes-for-examples (#10387)
  Implement gizmo presets and optimize code (#10257)
  scene name validation on save as and rename (#10347)
  prevent scene name overflow (#10400)
  App core tech debt componentization (#10395)
  add dependency to hierarchy generation that updates with every sourced entity created (#10401)
  hot fix install projects query (#10398)
  Support ReactNode in ModalHeader (#10391)
  [IR-2545] studio: Assets Panel ContextMenu (#10390)
  Removed FullscreenContainer, replaced with fullscreen on document.body (#10352)
  Ir 2564 fix fog (#10394)
  fix regressions

# Conflicts:
#	packages/ui/src/components/editor/panels/Files/container/index.tsx
MbfloydIR added a commit that referenced this pull request Jun 18, 2024
* dev:
  IR-1775-Changes-for-examples (#10387)
  Implement gizmo presets and optimize code (#10257)
  scene name validation on save as and rename (#10347)
  prevent scene name overflow (#10400)
  App core tech debt componentization (#10395)
  add dependency to hierarchy generation that updates with every sourced entity created (#10401)
  hot fix install projects query (#10398)
  Support ReactNode in ModalHeader (#10391)
  [IR-2545] studio: Assets Panel ContextMenu (#10390)
  Removed FullscreenContainer, replaced with fullscreen on document.body (#10352)
  Ir 2564 fix fog (#10394)
  fix regressions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants