Update miro extension by Silv-1 · Pull Request #17727 · raycast/extensions

Silv-1 · 2025-03-10T13:18:01Z

Description

Screencast

Checklist

I read the extension guidelines
I read the documentation about publishing
I ran npm run build and tested this distribution build in Raycast
I checked that files in the assets folder are used by the extension itself
I checked that assets used by the README are placed outside of the metadata folder

- Remove unused MiroApi - Update package.json publish command - Remove owner - Add owner - Add command to list recent boards - Initial commit

raycastbot · 2025-03-10T13:18:25Z

Thank you for your contribution! 🎉

🔔 @felixhaeberle @mathieudutour you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

Due to our current reduced availability, the initial review may take up to 10-15 business days

greptile-apps

PR Summary

This PR adds a new "Recent Boards" command to the Miro extension and reorganizes the board listing functionality with improved component structure.

The changelog entry for "Add command to show recent boards" should use {PR_MERGE_DATE} instead of "2025-03-10"
The command name change from list-boards to list-my-boards in package.json may affect users' saved preferences - consider keeping the old name
authorize() calls in list-my-boards.tsx and list-recent-boards.tsx should be wrapped in try-catch blocks
Could simplify error toast handling in list-boards.tsx by using showFailureToast from @raycast/utils
Since there are multiple commands, consider adding subtitles with "Miro" to differentiate them in the root search

_{💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!}

_{6 file(s) reviewed, 7 comment(s)}
_{Edit PR Review Bot Settings | Greptile}

extensions/miro/package.json

greptile-apps · 2025-03-10T13:19:12Z

extensions/miro/package.json

+    {
+      "name": "list-recent-boards",
+      "title": "Recent Boards",
+      "description": "Search & Open recent Miro boards in seconds",
+      "mode": "view"
    }


style: Consider adding a subtitle with 'Miro' to distinguish commands when multiple extensions are shown together

I followed the previous structure of descriptions.

greptile-apps · 2025-03-10T13:19:23Z

extensions/miro/src/components/list-boards.tsx

+                              toast.style = Toast.Style.Success;
+                            } catch (err) {
+                              console.error(err);
+                              toast.title = "Could not remove member.";


logic: Error message 'Could not remove member' is incorrect for board deletion operation

Suggested change

toast.title = "Could not remove member.";

toast.title = "Could not delete board.";

Did not touch this implementation, only moved it.

greptile-apps · 2025-03-10T13:19:23Z

extensions/miro/src/components/list-boards.tsx

+                            const toast = await showToast({ style: Toast.Style.Animated, title: "Deleting board..." });
+                            try {
+                              await mutate(miro.deleteBoard(item.id), {
+                                optimisticUpdate(data) {
+                                  return data.filter((x) => x.id !== item.id);
+                                },
+                              });
+                              toast.title = "🎉 Board deleted!";
+                              toast.style = Toast.Style.Success;
+                            } catch (err) {
+                              console.error(err);
+                              toast.title = "Could not remove member.";
+                              toast.message = String(err);
+                              toast.style = Toast.Style.Failure;
+                            }


style: Could be simplified using showFailureToast from @raycast/utils

Did not touch this implementation, only moved it.

extensions/miro/CHANGELOG.md

greptile-apps · 2025-03-10T13:20:14Z

extensions/miro/src/list-recent-boards.tsx

+    async () => {
+      await miro.authorize();
+      return await miro.fetchRecentItems();
+    },


logic: API calls should be wrapped in try-catch to handle potential auth and network errors gracefully

Suggested change

async () => {

await miro.authorize();

return await miro.fetchRecentItems();

},

async () => {

try {

await miro.authorize();

return await miro.fetchRecentItems();

} catch (error) {

showFailureToast(error instanceof Error ? error.message : "Failed to fetch recent boards");

return [];

}

},

Copied the implementation from list-boards.tsx and adapted it for recent boards. Did not do any refactoring.

greptile-apps · 2025-03-10T13:20:44Z

extensions/miro/src/oauth/miro.tsx

+async function fetchItems(url: RequestInfo): Promise<Board[]> {
+  const response = await fetch(url, {


logic: fetchItems should call authorize() first to ensure valid auth token

Suggested change

async function fetchItems(url: RequestInfo): Promise<Board[]> {

const response = await fetch(url, {

async function fetchItems(url: RequestInfo): Promise<Board[]> {

await authorize();

const response = await fetch(url, {

Did not touch this implementation, only moved it.

pernielsentikaer · 2025-03-14T12:12:20Z

Looks good to me, @felixhaeberle do you want to check this?

Silv-1 · 2025-03-26T13:23:42Z

@pernielsentikaer would you consider approving, or do you think we should wait a bit longer?

felixhaeberle · 2025-03-26T21:12:08Z

THANKS @Silv-1 for taking this over <3

pernielsentikaer

Hi 👋

Looks good to me, approved 🔥

github-actions · 2025-03-27T12:18:16Z

Published to the Raycast Store:
https://raycast.com/felixhaeberle/miro

raycastbot · 2025-03-27T12:18:18Z

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

Update miro extension

04a6649

- Remove unused MiroApi - Update package.json publish command - Remove owner - Add owner - Add command to list recent boards - Initial commit

Silv-1 requested a review from mathieudutour as a code owner March 10, 2025 13:18

raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: miro Issues related to the miro extension labels Mar 10, 2025

greptile-apps bot reviewed Mar 10, 2025

View reviewed changes

Silv-1 and others added 5 commits March 10, 2025 14:24

Delete extensions/miro/src/list-boards.tsx

c98f339

Rename command back to list-boards

52c242d

Rename list-my-boards.tsx to list-boards.tsx

786ebba

Use PR_MERGE_DATE

2ca4f15

Update package.json

dbc00bd

pernielsentikaer self-assigned this Mar 14, 2025

pernielsentikaer approved these changes Mar 27, 2025

View reviewed changes

Update CHANGELOG.md and optimise images

40d8cb1

raycastbot merged commit 6d4a2e7 into raycast:main Mar 27, 2025
2 checks passed

Silv-1 deleted the ext/miro branch April 21, 2025 13:43

Silv-1 mentioned this pull request May 22, 2025

Update miro extension #18702

Merged

5 tasks

	toast.title = "Could not remove member.";
	toast.title = "Could not delete board.";

		async function fetchItems(url: RequestInfo): Promise<Board[]> {
		const response = await fetch(url, {

Conversation

Silv-1 commented Mar 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Screencast

Checklist

Uh oh!

raycastbot commented Mar 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

PR Summary

Uh oh!

Uh oh!

greptile-apps bot Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

Silv-1 Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

Silv-1 Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

Silv-1 Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

greptile-apps bot Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

Silv-1 Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

Silv-1 Mar 10, 2025

Choose a reason for hiding this comment

Uh oh!

pernielsentikaer commented Mar 14, 2025

Uh oh!

Silv-1 commented Mar 26, 2025

Uh oh!

felixhaeberle commented Mar 26, 2025

Uh oh!

pernielsentikaer left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions bot commented Mar 27, 2025

Uh oh!

raycastbot commented Mar 27, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Silv-1 commented Mar 10, 2025 •

edited

Loading

raycastbot commented Mar 10, 2025 •

edited

Loading