Closed elisadinsmore closed 5 months ago
elisadinsmore
commented
7 months ago @ashleygmulligan2 the links I changed in this PR are invalid on the page itself, they don't redirect properly: https://flatfile.com/docs/apps/guides/egress
But I'm not actually sure how to modify them with the snippet format because they look like they're correct, but aren't working correctly 🤔
coderabbitai[bot]
commented
5 months ago The recent update focuses on refining the handling of data during change events and updating links related to theming and egress actions in our code snippets. These changes aim to improve the functionality and accessibility of our code examples, making it easier for users to understand and implement the provided solutions effectively.
| Files | Changes Summary |
|---|---|
_snippets/shared/embedded_listener_basicreact.mdx |
Removed handling data references for change events; updated links for theming and egress in snippets. |
_snippets/shared/embedded_listener_basicreact.mdx (34)
Near line 6: Possible spelling mistake found. Context: ...e with this simple Blueprint. 2. Update `App.tsx` to import the Workbook. Next time you... --- Near line 8: Possible spelling mistake found. Context: ...mport the Workbook. Next time you open Flatfile, you'll see a Workbook called "Contacts... --- Near line 8: Only proper nouns start with an uppercase character (there are exceptions for headlines). Context: ...Workbook called "Contacts" that has one Sheet with three fields. Your Workbook will a... --- Near line 13: This sentence does not start with an uppercase letter. Context: ...ort { Flatfile } from "@flatfile/api"; export const workbook: Flatfile.CreateWorkbook... --- Near line 14: Possible spelling mistake found. Context: ...latfile } from "@flatfile/api"; export const workbook: Flatfile.CreateWorkbookConfig... --- Near line 14: Possible spelling mistake found. Context: ...@flatfile/api"; export const workbook: Flatfile.CreateWorkbookConfig = { name: "All D... --- Near line 14: Add a space between sentences. Context: .../api"; export const workbook: Flatfile.CreateWorkbookConfig = { name: "All Data", labels: ["pin... --- Near line 14: Don’t put a space after the opening parenthesis. Context: ...rkbook: Flatfile.CreateWorkbookConfig = { name: "All Data", labels: ["pinned"]... --- Near line 17: Don’t put a space after the opening parenthesis. Context: ... Data", labels: ["pinned"], sheets: [ { name: "Contacts", slug... --- Near line 18: Don’t put a space after the opening parenthesis. Context: ..., labels: ["pinned"], sheets: [ { name: "Contacts", slug: "con... --- Near line 21: Don’t put a space after the opening parenthesis. Context: ..., slug: "contacts", fields: [ { key: "firstName", ... --- Near line 22: Don’t put a space after the opening parenthesis. Context: ...ug: "contacts", fields: [ { key: "firstName", ty... --- Near line 23: Possible spelling mistake found. Context: ... fields: [ { key: "firstName", type: "string", l... --- Near line 26: Don’t put a space before the closing parenthesis. Context: ..., label: "First Name", }, { key: "lastName", ... --- Near line 27: Don’t put a space after the opening parenthesis. Context: ...label: "First Name", }, { key: "lastName", typ... --- Near line 28: Possible spelling mistake found. Context: ...", }, { key: "lastName", type: "string", l... --- Near line 31: Don’t put a space before the closing parenthesis. Context: ...", label: "Last Name", }, { key: "email", ... --- Near line 32: Don’t put a space after the opening parenthesis. Context: ... label: "Last Name", }, { key: "email", type: ... --- Near line 36: Don’t put a space before the closing parenthesis. Context: ...ring", label: "Email", }, ], }, ], actions: [ ... --- Near line 37: Don’t put a space before the closing parenthesis. Context: ... label: "Email", }, ], }, ], actions: [ { o... --- Near line 38: Don’t put a space before the closing parenthesis. Context: ... label: "Email", }, ], }, ], actions: [ { operatio... --- Near line 39: Don’t put a space before the closing parenthesis. Context: ...l: "Email", }, ], }, ], actions: [ { operation: "s... --- Near line 40: Don’t put a space after the opening parenthesis. Context: ... }, ], }, ], actions: [ { operation: "submitActionFg",... --- Near line 41: Don’t put a space after the opening parenthesis. Context: ..., ], }, ], actions: [ { operation: "submitActionFg", ... --- Near line 42: Possible spelling mistake found. Context: ...], actions: [ { operation: "submitActionFg", mode: "foreground", label... --- Near line 47: Don’t put a space before the closing parenthesis. Context: ... webhook.site", primary: true, }, ], }; ``` ```tsx src/App.tsx import... --- Near line 48: Don’t put a space before the closing parenthesis. Context: ...ook.site", primary: true, }, ], }; ``` ```tsx src/App.tsx import Reac... --- Near line 48: Don’t put a space before the closing parenthesis. Context: ....site", primary: true, }, ], }; ``` ```tsx src/App.tsx import React, ... --- Near line 113: Possible spelling mistake found. Context: ...`src/listener.ts` file with this simple `recordHook`. 2. Update `App.tsx` to import the lis... --- Near line 114: Possible spelling mistake found. Context: ...ith this simple `recordHook`. 2. Update `App.tsx` to import the listener. Once you add ... --- Near line 205: Possible spelling mistake found. Context: ...### 6. Match your brand By attaching a `themeConfig` to `flatfileOptions` in `src/App.tsx`,... --- Near line 205: Possible spelling mistake found. Context: ... brand By attaching a `themeConfig` to `flatfileOptions` in `src/App.tsx`, we will now override... --- Near line 205: Possible spelling mistake found. Context: ...a `themeConfig` to `flatfileOptions` in `src/App.tsx`, we will now override colors in your S... --- Near line 205: Consider removing “of” to be more concise Context: ... in your Space to match your brand. See all of the options here in the [Theming Reference]...
_snippets/shared/embedded_listener_basicreact.mdx (3)
`205-205`: Verify the updated theming link to ensure it redirects correctly. Please check the link processing mechanism as there might be an underlying issue affecting redirection. --- `238-238`: Verify the updated egress link to ensure it redirects correctly. Please check the link processing mechanism as there might be an underlying issue affecting redirection. --- `116-116`: Removal of data handling on change events appears to simplify event management. Please ensure thorough testing to confirm that this change does not negatively impact the existing functionalities.
Updated links to other pages - these links were previously invalid
Summary by CodeRabbit