sanity icon indicating copy to clipboard operation
sanity copied to clipboard

fix(scheduled-publishing): don't include it if it's the only plugin available

Open pedrobonamin opened this issue 9 months ago • 5 comments

Before the implementation of scheduled publishing as a default feature, when visiting the studio with no plugins added users were seeing the following screen:

Screenshot 2024-04-30 at 13 05 32

Once scheduled publishing was implemented, users no longer see that screen and instead are seeing the schedule publishing tool. This PR fixes that, and only includes into the plugins array the scheduled publishing tool if other plugins have been added to the source.

Description

What to review

Testing

Notes for release

pedrobonamin avatar Apr 30 '24 11:04 pedrobonamin

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
performance-studio ✅ Ready (Inspect) Visit Preview May 6, 2024 4:44pm
test-next-studio ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2024 4:44pm
test-studio ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2024 4:44pm
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
studio-workshop ⬜️ Ignored (Inspect) Visit Preview May 6, 2024 4:44pm

vercel[bot] avatar Apr 30 '24 11:04 vercel[bot]

No changes to documentation

github-actions[bot] avatar Apr 30 '24 11:04 github-actions[bot]

Component Testing Report Updated May 6, 2024 4:49 PM (UTC)

File Status Duration Passed Skipped Failed
comments/CommentInput.spec.tsx ✅ Passed (Inspect) 39s 15 0 0
formBuilder/ArrayInput.spec.tsx ✅ Passed (Inspect) 6s 3 0 0
formBuilder/inputs/PortableText/Annotations.spec.tsx ✅ Passed (Inspect) 26s 6 0 0
formBuilder/inputs/PortableText/copyPaste/CopyPaste.spec.tsx ✅ Passed (Inspect) 20s 6 3 0
formBuilder/inputs/PortableText/Decorators.spec.tsx ✅ Passed (Inspect) 14s 6 0 0
formBuilder/inputs/PortableText/FocusTracking.spec.tsx ✅ Passed (Inspect) 36s 15 0 0
formBuilder/inputs/PortableText/Input.spec.tsx ✅ Passed (Inspect) 1m 2s 14 0 0
formBuilder/inputs/PortableText/ObjectBlock.spec.tsx ✅ Passed (Inspect) 1m 5s 18 0 0
formBuilder/inputs/PortableText/PresenceCursors.spec.tsx ✅ Passed (Inspect) 7s 3 9 0
formBuilder/inputs/PortableText/RangeDecoration.spec.tsx ✅ Passed (Inspect) 21s 9 0 0
formBuilder/inputs/PortableText/Styles.spec.tsx ✅ Passed (Inspect) 15s 6 0 0
formBuilder/inputs/PortableText/Toolbar.spec.tsx ✅ Passed (Inspect) 22s 9 0 0

github-actions[bot] avatar Apr 30 '24 11:04 github-actions[bot]

Just a thought on whether there are any tests we can add around this @pedrobonamin ?

jordanl17 avatar May 06 '24 11:05 jordanl17

Just a thought on whether there are any tests we can add around this @pedrobonamin ?

Thanks @jordanl17 added some tests for this.

pedrobonamin avatar May 06 '24 16:05 pedrobonamin