djkato/saleor-app-sdk-REDIS_APL
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Added RequestPermission event (#261)

Browse source 
* add action

* add event

* changeset

* Remove eve

* remove event

* Cleanup

* remove commnets
This commit is contained in:
Lukasz Ostrowski 2023-07-25 09:11:34 +02:00 committed by GitHub
parent 8eae1614f2
commit 1118ea9c0b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 57 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
.changeset/grumpy-seals-fail.md Normal file
View file

@ -0,0 +1,11 @@
---
"@saleor/app-sdk": minor
---
Added new actions and events for dynamic permissions request.
Now App can ask Dashboard to grant more permissions than originally assigned.
Operation can be approved or rejected by the user.
This feature is available in Saleor 3.15 and higher

21
docs/app-bridge.md
View file

@ -176,13 +176,14 @@ handleRedirect();
### Available actions ### Available actions
| Action | Arguments | Description | | Action | Arguments | Description |
| :-------------- | :--------------------------------------------------------------- | :--------------------------------------- | | :---------------------------- | :----------------------------------------------------------------------------------------------------------- | :------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
| `Redirect` | `to` (string) - relative (inside Dashboard) or absolute URL path | | | `Redirect` | `to` (string) - relative (inside Dashboard) or absolute URL path | |
| | `newContext` (boolean) - should open in a new browsing context | | | | `newContext` (boolean) - should open in a new browsing context | |
| `Notification` | `status` (`info` / `success` / `warning` / `error` / undefined) | | | `Notification` | `status` (`info` / `success` / `warning` / `error` / undefined) | |
| | `title` (string / undefined) - title of the notification | | | | `title` (string / undefined) - title of the notification | |
| | `text` (string / undefined) - content of the notification | | | | `text` (string / undefined) - content of the notification | |
| | `apiMessage` (string / undefined) - error log from api | | | | `apiMessage` (string / undefined) - error log from api | |
| `NotifyReady` | | Inform Dashboard that AppBridge is ready | | `NotifyReady` | | Inform Dashboard that AppBridge is ready |
| `UpdateRouting` | `newRoute` - current path of App to be set in URL | | | `UpdateRouting` | `newRoute` - current path of App to be set in URL | |
| `RequestPermissions` (>=3.15) | `permissions` - array of AppPermission, `redirectPath` - path app will be redirected to after operation ends | Ask Dashboard to give more permissions to the app. Dashboard will unmount app. After user approves or denies, Dashboard will redirect to `redirectPath`. If operation fails, `?error=REASON` will be appended |

36
src/app-bridge/actions.ts
View file

@ -1,5 +1,6 @@
import { v4 as uuidv4 } from "uuid"; import { v4 as uuidv4 } from "uuid";
import { AppPermission } from "../types";
import { Values } from "./helpers"; import { Values } from "./helpers";
// Using constants over Enums, more info: https://fettblog.eu/tidy-typescript-avoid-enums/ // Using constants over Enums, more info: https://fettblog.eu/tidy-typescript-avoid-enums/
@ -20,6 +21,12 @@ export const ActionType = {
* Inform Dashboard that AppBridge is ready * Inform Dashboard that AppBridge is ready
*/ */
notifyReady: "notifyReady", notifyReady: "notifyReady",
/**
* Request one or more permissions from the Dashboard
*
* Available from 3.15
*/
requestPermission: "requestPermissions",
} as const; } as const;
export type ActionType = Values<typeof ActionType>; export type ActionType = Values<typeof ActionType>;
@ -110,11 +117,38 @@ function createNotifyReadyAction(): NotifyReady {
}); });
} }
export type Actions = RedirectAction | NotificationAction | UpdateRouting | NotifyReady; export type RequestPermissions = ActionWithId<
"requestPermissions",
{
permissions: AppPermission[];
redirectPath: string;
}
>;
function createRequestPermissionsAction(
permissions: AppPermission[],
redirectPath: string
): RequestPermissions {
return withActionId({
type: "requestPermissions",
payload: {
permissions,
redirectPath,
},
});
}
export type Actions =
| RedirectAction
| NotificationAction
| UpdateRouting
| NotifyReady
| RequestPermissions;
export const actions = { export const actions = {
Redirect: createRedirectAction, Redirect: createRedirectAction,
Notification: createNotificationAction, Notification: createNotificationAction,
UpdateRouting: createUpdateRoutingAction, UpdateRouting: createUpdateRoutingAction,
NotifyReady: createNotifyReadyAction, NotifyReady: createNotifyReadyAction,
RequestPermissions: createRequestPermissionsAction,
}; };

2
src/app-bridge/app-bridge.ts
View file

@ -19,8 +19,6 @@ type SubscribeMap = {
const debug = debugPkg.debug("app-sdk:AppBridge"); const debug = debugPkg.debug("app-sdk:AppBridge");
function eventStateReducer(state: AppBridgeState, event: Events) { function eventStateReducer(state: AppBridgeState, event: Events) {
debug("Event reducer received event: %j", event);
switch (event.type) { switch (event.type) {
case EventType.handshake: { case EventType.handshake: {
const userJwtPayload = extractUserFromJwt(event.payload.token); const userJwtPayload = extractUserFromJwt(event.payload.token);