feat: add edit popup on sponsor pages list #770
+197
−16
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
📝 WalkthroughWalkthroughThe PR enhances sponsor page management by introducing timezone support for page templates, adding new action creators for single sponsor page retrieval and persistence, and integrating a page template-based creation workflow. State management is extended to track current sponsor pages and summit timezone throughout the component lifecycle. Changes
Sequence Diagram(s)sequenceDiagram
actor User
participant SponsorPagesListPage as SponsorPagesListPage<br/>(Component)
participant PageTemplatePopup as PageTemplatePopup<br/>(Component)
participant SponsorPageActions as sponsor-pages-actions<br/>(Actions)
participant API as API<br/>(Server)
participant SponsorPagesReducer as sponsor-pages-list-reducer<br/>(State)
User->>SponsorPagesListPage: Click edit/create page
SponsorPagesListPage->>SponsorPageActions: getSponsorPage(pageId) or<br/>show PageTemplatePopup
SponsorPageActions->>API: GET /sponsor-pages/:id
API-->>SponsorPageActions: sponsor page data
SponsorPageActions->>SponsorPageActions: normalizeSponsorPage()<br/>(convert deadlines,<br/>normalize file_type)
SponsorPageActions->>SponsorPagesReducer: dispatch RECEIVE_SPONSOR_PAGE
SponsorPagesReducer-->>SponsorPagesListPage: currentSponsorPage updated
SponsorPagesListPage->>PageTemplatePopup: pass currentSponsorPage<br/>as pageTemplate
User->>PageTemplatePopup: modify and save
PageTemplatePopup->>SponsorPagesListPage: onSave(entity)
SponsorPagesListPage->>SponsorPageActions: saveSponsorPage(entity)
SponsorPageActions->>SponsorPageActions: normalizeSponsorPage()
SponsorPageActions->>API: PUT/POST /sponsor-pages
API-->>SponsorPageActions: success
SponsorPageActions->>SponsorPagesReducer: dispatch SPONSOR_PAGE_UPDATED<br/>or SPONSOR_PAGE_ADDED
SponsorPageActions->>SponsorPagesListPage: refresh page list
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~22 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 5
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/reducers/sponsors/sponsor-pages-list-reducer.js (1)
46-57:⚠️ Potential issue | 🟡 MinorAvoid clobbering
summitTZwhen payload lacks it.
getSponsorPagesdoesn’t currently supplysummitTZ, so this resets the stored value toundefined.🛠️ Suggested fix
- const { order, orderDir, page, term, hideArchived, summitTZ } = payload; + const { order, orderDir, page, term, hideArchived, summitTZ } = payload; return { ...state, order, orderDir, sponsorPages: [], currentPage: page, term, hideArchived, - summitTZ + summitTZ: summitTZ ?? state.summitTZ };
🤖 Fix all issues with AI agents
In `@src/actions/page-template-actions.js`:
- Around line 57-60: The code dereferences currentSummit.time_zone.name into
summitTZ without guarding for missing currentSummit or time_zone; update the
async action (the function using dispatch,getState) to first check that
currentSummit and currentSummit.time_zone exist and use a safe fallback (e.g.,
'UTC' or null) before reading .name, or bail out early if unset; apply the same
guard to the other occurrence around the code referenced at the second spot (the
other async action/usage at the later location) so the list fetch won't throw
when currentSummit or time_zone is null.
In `@src/actions/sponsor-pages-actions.js`:
- Around line 126-152: normalizeSponsorPage currently assumes entity.modules
exists and calls .map which throws if modules is undefined; update
normalizeSponsorPage to guard against missing modules by checking entity.modules
(or using a default empty array) before mapping and assign
normalizedEntity.modules accordingly so drafts/new entities without modules
don't crash; reference the normalizeSponsorPage function and the entity.modules
usage when making this change.
- Around line 104-124: The getSponsorPage thunk currently calls
dispatch(stopLoading()) only in the success .then() branch so the loading
spinner can remain if the fetch fails; update getSponsorPage to ensure
stopLoading() always runs by appending a .finally() (or using try/finally if you
convert to async/await) to the promise returned by getRequest (the call that
returns a function invoked with (params)(dispatch)), so that
dispatch(stopLoading()) is executed in the .finally() handler; keep the existing
dispatch(startLoading()) at the top and preserve authErrorHandler and
RECEIVE_SPONSOR_PAGE usage.
In `@src/pages/sponsors-global/page-templates/page-template-popup.js`:
- Around line 118-133: The normalizeModules function (and PageTemplatePopup)
currently rely on a summitTZ parameter that is undefined because
pageTemplateListState lacks summitTZ; add summitTZ to the
pageTemplateListReducer's state (and initialize it, e.g., to "UTC" or the real
summit timezone) and ensure the selector that builds pageTemplateListState
includes this summitTZ so PageTemplatePopup receives a defined value, or
alternatively explicitly coalesce summitTZ at the call site when passing into
normalizeModules/PageTemplatePopup (e.g., pass state.summitTZ || "UTC"); update
references to summitTZ in the reducer/selector and any component props
(pageTemplateListState) to keep behavior explicit and consistent with
sponsor-page-forms-list-reducer.
In `@src/pages/sponsors/sponsor-pages-list-page/index.js`:
- Around line 75-78: The popup uses the same "new" state and shared
currentSponsorPage for both creating and editing, so opening "new" after an edit
can reuse the existing id and overwrite; update the flow so edit and create use
distinct modes (e.g., setOpenPopup("edit") in handleRowEdit and
setOpenPopup("new") only for creation) or explicitly clear/reset
currentSponsorPage when opening the "new" popup; change handleRowEdit to set an
"edit" mode (or set a separate isEditing flag) and ensure any code that opens
the create popup (the other place where setOpenPopup("new") is called) resets
currentSponsorPage/template to an empty object so new saves create instead of
update.
Comment on lines
+57
to
+60
| async (dispatch, getState) => { | ||
| const { currentSummitState } = getState(); | ||
| const { currentSummit } = currentSummitState; | ||
| const summitTZ = currentSummit.time_zone.name; |
There was a problem hiding this comment.
Guard against missing summit time zone before dereferencing.
currentSummit or time_zone can be null during initial load; this would throw and break the list fetch. Add a safe fallback.
🛠️ Suggested fix
- const { currentSummit } = currentSummitState;
- const summitTZ = currentSummit.time_zone.name;
+ const { currentSummit } = currentSummitState || {};
+ const summitTZ = currentSummit?.time_zone?.name ?? "UTC";Also applies to: 98-98
🤖 Prompt for AI Agents
In `@src/actions/page-template-actions.js` around lines 57 - 60, The code
dereferences currentSummit.time_zone.name into summitTZ without guarding for
missing currentSummit or time_zone; update the async action (the function using
dispatch,getState) to first check that currentSummit and currentSummit.time_zone
exist and use a safe fallback (e.g., 'UTC' or null) before reading .name, or
bail out early if unset; apply the same guard to the other occurrence around the
code referenced at the second spot (the other async action/usage at the later
location) so the list fetch won't throw when currentSummit or time_zone is null.
Comment on lines
+104
to
+124
| export const getSponsorPage = (pageId) => async (dispatch, getState) => { | ||
| const { currentSummitState } = getState(); | ||
| const { currentSummit } = currentSummitState; | ||
| const accessToken = await getAccessTokenSafely(); | ||
|
|
||
| dispatch(startLoading()); | ||
|
|
||
| const params = { | ||
| access_token: accessToken, | ||
| expand: "modules.file_type" | ||
| }; | ||
|
|
||
| return getRequest( | ||
| null, | ||
| createAction(RECEIVE_SPONSOR_PAGE), | ||
| `${window.SPONSOR_PAGES_API_URL}/api/v1/summits/${currentSummit.id}/show-pages/${pageId}`, | ||
| authErrorHandler | ||
| )(params)(dispatch).then(() => { | ||
| dispatch(stopLoading()); | ||
| }); | ||
| }; |
There was a problem hiding this comment.
Ensure loading state clears on failed fetch.
stopLoading() only runs in the success path; on errors the spinner can get stuck. Use finally to guarantee cleanup.
🛠️ Suggested fix
- return getRequest(
+ return getRequest(
null,
createAction(RECEIVE_SPONSOR_PAGE),
`${window.SPONSOR_PAGES_API_URL}/api/v1/summits/${currentSummit.id}/show-pages/${pageId}`,
authErrorHandler
- )(params)(dispatch).then(() => {
- dispatch(stopLoading());
- });
+ )(params)(dispatch)
+ .finally(() => {
+ dispatch(stopLoading());
+ });🤖 Prompt for AI Agents
In `@src/actions/sponsor-pages-actions.js` around lines 104 - 124, The
getSponsorPage thunk currently calls dispatch(stopLoading()) only in the success
.then() branch so the loading spinner can remain if the fetch fails; update
getSponsorPage to ensure stopLoading() always runs by appending a .finally() (or
using try/finally if you convert to async/await) to the promise returned by
getRequest (the call that returns a function invoked with (params)(dispatch)),
so that dispatch(stopLoading()) is executed in the .finally() handler; keep the
existing dispatch(startLoading()) at the top and preserve authErrorHandler and
RECEIVE_SPONSOR_PAGE usage.
Comment on lines
+126
to
+152
| const normalizeSponsorPage = (entity) => { | ||
| const normalizedEntity = { ...entity }; | ||
|
|
||
| normalizedEntity.modules = entity.modules.map((module) => { | ||
| const normalizedModule = { ...module }; | ||
|
|
||
| if (module.kind === PAGES_MODULE_KINDS.MEDIA && module.upload_deadline) { | ||
| normalizedModule.upload_deadline = moment | ||
| .utc(module.upload_deadline) | ||
| .unix(); | ||
| } | ||
|
|
||
| if (module.kind === PAGES_MODULE_KINDS.MEDIA && module.file_type_id) { | ||
| normalizedModule.file_type_id = | ||
| module.file_type_id?.value || module.file_type_id; | ||
| } | ||
|
|
||
| if (module.kind === PAGES_MODULE_KINDS.DOCUMENT && module.file) { | ||
| normalizedModule.file = module.file[0] || null; | ||
| } | ||
|
|
||
| delete normalizedModule._tempId; | ||
|
|
||
| return normalizedModule; | ||
| }); | ||
|
|
||
| return normalizedEntity; |
There was a problem hiding this comment.
Defend against missing modules in normalization.
If entity.modules is undefined (e.g., draft/new), .map will throw.
🛠️ Suggested fix
- normalizedEntity.modules = entity.modules.map((module) => {
+ const modules = entity.modules ?? [];
+ normalizedEntity.modules = modules.map((module) => {📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| const normalizeSponsorPage = (entity) => { | |
| const normalizedEntity = { ...entity }; | |
| normalizedEntity.modules = entity.modules.map((module) => { | |
| const normalizedModule = { ...module }; | |
| if (module.kind === PAGES_MODULE_KINDS.MEDIA && module.upload_deadline) { | |
| normalizedModule.upload_deadline = moment | |
| .utc(module.upload_deadline) | |
| .unix(); | |
| } | |
| if (module.kind === PAGES_MODULE_KINDS.MEDIA && module.file_type_id) { | |
| normalizedModule.file_type_id = | |
| module.file_type_id?.value || module.file_type_id; | |
| } | |
| if (module.kind === PAGES_MODULE_KINDS.DOCUMENT && module.file) { | |
| normalizedModule.file = module.file[0] || null; | |
| } | |
| delete normalizedModule._tempId; | |
| return normalizedModule; | |
| }); | |
| return normalizedEntity; | |
| const normalizeSponsorPage = (entity) => { | |
| const normalizedEntity = { ...entity }; | |
| const modules = entity.modules ?? []; | |
| normalizedEntity.modules = modules.map((module) => { | |
| const normalizedModule = { ...module }; | |
| if (module.kind === PAGES_MODULE_KINDS.MEDIA && module.upload_deadline) { | |
| normalizedModule.upload_deadline = moment | |
| .utc(module.upload_deadline) | |
| .unix(); | |
| } | |
| if (module.kind === PAGES_MODULE_KINDS.MEDIA && module.file_type_id) { | |
| normalizedModule.file_type_id = | |
| module.file_type_id?.value || module.file_type_id; | |
| } | |
| if (module.kind === PAGES_MODULE_KINDS.DOCUMENT && module.file) { | |
| normalizedModule.file = module.file[0] || null; | |
| } | |
| delete normalizedModule._tempId; | |
| return normalizedModule; | |
| }); | |
| return normalizedEntity; | |
| }; |
🤖 Prompt for AI Agents
In `@src/actions/sponsor-pages-actions.js` around lines 126 - 152,
normalizeSponsorPage currently assumes entity.modules exists and calls .map
which throws if modules is undefined; update normalizeSponsorPage to guard
against missing modules by checking entity.modules (or using a default empty
array) before mapping and assign normalizedEntity.modules accordingly so
drafts/new entities without modules don't crash; reference the
normalizeSponsorPage function and the entity.modules usage when making this
change.
Comment on lines
+118
to
+133
| const normalizeModules = (modules = [], summitTZ = "UTC") => | ||
| modules.map((m) => { | ||
| if (m.kind === PAGES_MODULE_KINDS.MEDIA) { | ||
| const normalizeModule = { ...m }; | ||
| if (m.upload_deadline) { | ||
| normalizeModule.upload_deadline = epochToMomentTimeZone( | ||
| m.upload_deadline, | ||
| summitTZ | ||
| ); | ||
| } | ||
| if (m.file_type) { | ||
| normalizeModule.file_type_id = { | ||
| value: m.file_type.id, | ||
| label: `${m.file_type.name} (${m.file_type.allowed_extensions})` | ||
| }; | ||
| } |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# First, find the file and check the actual code at the specified lines
cat -n "src/pages/sponsors-global/page-templates/page-template-popup.js" | sed -n '110,140p'Repository: fntechgit/summit-admin
Length of output: 1121
🏁 Script executed:
# Search for how normalizeModules is called throughout the codebase
rg "normalizeModules\(" --max-count=20Repository: fntechgit/summit-admin
Length of output: 200
🏁 Script executed:
# Find the epochToMomentTimeZone function definition
rg "epochToMomentTimeZone" -A 5 --max-count=10Repository: fntechgit/summit-admin
Length of output: 50378
🏁 Script executed:
# Find where summitTZ is defined/used in the page-template-popup.js file
grep -n "summitTZ" "src/pages/sponsors-global/page-templates/page-template-popup.js" | head -20Repository: fntechgit/summit-admin
Length of output: 246
🏁 Script executed:
# Look at the component definition and props to understand summitTZ source
sed -n '1,150p' "src/pages/sponsors-global/page-templates/page-template-popup.js" | grep -A 5 -B 5 "summitTZ\|export\|const\|function"Repository: fntechgit/summit-admin
Length of output: 3546
🏁 Script executed:
# Find parent components that use PageTemplatePopup to see how summitTZ is passed
rg "PageTemplatePopup" --max-count=10Repository: fntechgit/summit-admin
Length of output: 817
🏁 Script executed:
# Check if there are any null checks or guards for summitTZ elsewhere in similar components
rg "summitTZ\s*\|\|" --max-count=10Repository: fntechgit/summit-admin
Length of output: 48
🏁 Script executed:
# Check how summitTZ is passed in page-template-list-page.js
grep -A 10 "PageTemplatePopup" "src/pages/sponsors-global/page-templates/page-template-list-page.js" | head -20Repository: fntechgit/summit-admin
Length of output: 464
🏁 Script executed:
# Check how summitTZ is passed in sponsor-pages-list-page
grep -A 10 "PageTemplatePopup" "src/pages/sponsors/sponsor-pages-list-page/index.js" | head -20Repository: fntechgit/summit-admin
Length of output: 501
🏁 Script executed:
# Check where summitTZ comes from in page-template-list-page.js
grep -B 20 "summitTZ={summitTZ}" "src/pages/sponsors-global/page-templates/page-template-list-page.js" | grep -E "summitTZ|const|useState|mapStateToProps"Repository: fntechgit/summit-admin
Length of output: 93
🏁 Script executed:
# Check the component definition and props for sponsor-pages-list-page to see why summitTZ is missing
head -100 "src/pages/sponsors/sponsor-pages-list-page/index.js" | tail -80Repository: fntechgit/summit-admin
Length of output: 1915
🏁 Script executed:
# Check if summitTZ is ever set to null anywhere in the component files
rg "summitTZ\s*=\s*null" --max-count=5Repository: fntechgit/summit-admin
Length of output: 48
🏁 Script executed:
# Check the mapStateToProps for page-template-list-page to see how summitTZ is sourced
grep -A 30 "mapStateToProps\|const.*summitTZ" "src/pages/sponsors-global/page-templates/page-template-list-page.js" | head -40Repository: fntechgit/summit-admin
Length of output: 332
🏁 Script executed:
# Find the page template list reducer to see what's in the state
fd "page-template-list" -t f | grep -i reducerRepository: fntechgit/summit-admin
Length of output: 127
🏁 Script executed:
# Search for where pageTemplateListState is defined/reduced
rg "pageTemplateListState|PAGE_TEMPLATE" --type js -l | head -5Repository: fntechgit/summit-admin
Length of output: 334
🏁 Script executed:
# Check the page-template-list-reducer to see the state structure and how summitTZ is set
cat -n "src/reducers/sponsors_inventory/page-template-list-reducer.js" | head -60Repository: fntechgit/summit-admin
Length of output: 2231
🏁 Script executed:
# Continue viewing the reducer to find where summitTZ comes from
cat -n "src/reducers/sponsors_inventory/page-template-list-reducer.js" | tail -60Repository: fntechgit/summit-admin
Length of output: 2221
🏁 Script executed:
# Also search for where summitTZ is imported or defined in store or root reducer
grep -r "summitTZ" "src/store.js" "src/reducers/index.js" 2>/dev/null | head -10Repository: fntechgit/summit-admin
Length of output: 48
🏁 Script executed:
# Check the full page-template-list-page.js to see where summitTZ comes from
cat -n "src/pages/sponsors-global/page-templates/page-template-list-page.js"Repository: fntechgit/summit-admin
Length of output: 9102
🏁 Script executed:
# Check the root reducer to see what state is available
cat -n "src/reducers/index.js" | head -80Repository: fntechgit/summit-admin
Length of output: 119
🏁 Script executed:
# Also check store.js to see the full state structure
cat -n "src/store.js"Repository: fntechgit/summit-admin
Length of output: 24864
🏁 Script executed:
# Check the currentSummitReducer to see if it has summitTZ
grep -n "summitTZ\|time_zone" "src/reducers/summits/current-summit-reducer.js" | head -20Repository: fntechgit/summit-admin
Length of output: 137
🏁 Script executed:
# Look at other pages that use summitTZ to see how they source it
grep -B 5 "summitTZ" "src/pages/sponsors/sponsor-pages-list-page/index.js" | head -20Repository: fntechgit/summit-admin
Length of output: 48
🏁 Script executed:
# Check if pageTemplateListReducer should have summitTZ by checking other similar list reducers
grep -l "summitTZ" "src/reducers/sponsors"/*.js | head -3Repository: fntechgit/summit-admin
Length of output: 228
🏁 Script executed:
# Get the actual state props passed to PageTemplateListPage by checking the full component
grep -A 50 "const PageTemplateListPage" "src/pages/sponsors-global/page-templates/page-template-list-page.js" | grep -E "summitTZ|currentSummit"Repository: fntechgit/summit-admin
Length of output: 76
🏁 Script executed:
# Check if summitTZ might come from currentSummitState instead
grep "currentSummit.*time_zone\|summitTZ" "src/reducers/summits/current-summit-reducer.js" | head -10Repository: fntechgit/summit-admin
Length of output: 48
🏁 Script executed:
# Check what summitTZ should actually be in sponsor-page-forms-list-reducer
grep -A 30 "DEFAULT_STATE\|summitTZ" "src/reducers/sponsors/sponsor-page-forms-list-reducer.js" | head -50Repository: fntechgit/summit-admin
Length of output: 1075
🏁 Script executed:
# Verify if PageTemplateListPage is actually receiving summitTZ correctly from Redux
grep "mapStateToProps\|summitTZ" "src/pages/sponsors-global/page-templates/page-template-list-page.js"Repository: fntechgit/summit-admin
Length of output: 204
🏁 Script executed:
# Check how summitTZ gets into sponsor-page-forms-list-reducer's REQUEST action
grep -B 10 "REQUEST_SPONSOR_MANAGED_FORMS\|getSponsorPages" "src/pages/sponsors-global/page-templates/page-template-list-page.js" | head -20Repository: fntechgit/summit-admin
Length of output: 48
🏁 Script executed:
# Check the actions to see if summitTZ is being passed to getPageTemplates
grep -A 20 "export.*getPageTemplates" "src/actions/page-template-actions.js" | head -30Repository: fntechgit/summit-admin
Length of output: 689
🏁 Script executed:
# Check if summitTZ is dispatched to pageTemplateListReducer
grep -n "pageTemplateListState\|REQUEST_PAGE_TEMPLATES\|RECEIVE_PAGE_TEMPLATES" "src/actions/page-template-actions.js" | head -20Repository: fntechgit/summit-admin
Length of output: 293
🏁 Script executed:
# Check the full action to see what's dispatched
cat -n "src/actions/page-template-actions.js" | head -80Repository: fntechgit/summit-admin
Length of output: 3355
Pass the timezone from state to avoid relying on default parameters.
The summitTZ prop passed to PageTemplatePopup is currently undefined because it's not included in pageTemplateListState. While the default parameter "UTC" catches undefined, this is fragile—it relies on accident rather than intent. For consistency with similar components (e.g., sponsor-page-forms-list-reducer), add summitTZ to pageTemplateListReducer's state so it's properly tracked and passed down, or explicitly coalesce it at the call site to make the behavior intentional.
🤖 Prompt for AI Agents
In `@src/pages/sponsors-global/page-templates/page-template-popup.js` around lines
118 - 133, The normalizeModules function (and PageTemplatePopup) currently rely
on a summitTZ parameter that is undefined because pageTemplateListState lacks
summitTZ; add summitTZ to the pageTemplateListReducer's state (and initialize
it, e.g., to "UTC" or the real summit timezone) and ensure the selector that
builds pageTemplateListState includes this summitTZ so PageTemplatePopup
receives a defined value, or alternatively explicitly coalesce summitTZ at the
call site when passing into normalizeModules/PageTemplatePopup (e.g., pass
state.summitTZ || "UTC"); update references to summitTZ in the reducer/selector
and any component props (pageTemplateListState) to keep behavior explicit and
consistent with sponsor-page-forms-list-reducer.
Comment on lines
75
to
78
| const handleRowEdit = (row) => { | ||
| getSponsorForm(row.id).then(() => { | ||
| getSponsorPage(row.id).then(() => { | ||
| setOpenPopup("new"); | ||
| }); |
There was a problem hiding this comment.
Edit vs. new mode can overwrite existing pages.
Both “edit” and “new” open the same popup state and reuse currentSponsorPage, so clicking “new” after an edit can retain id and update instead of creating. Separate modes or clear the template for new.
🛠️ Suggested fix
- const [openPopup, setOpenPopup] = useState(null);
+ const [popupMode, setPopupMode] = useState(null);
const handleRowEdit = (row) => {
getSponsorPage(row.id).then(() => {
- setOpenPopup("new");
+ setPopupMode("edit");
});
};
...
- onClick={() => setOpenPopup("new")}
+ onClick={() => setPopupMode("new")}
...
- <GlobalPagePopup
- open={openPopup === "clone"}
- onClose={() => setOpenPopup(null)}
- />
+ <GlobalPagePopup
+ open={popupMode === "clone"}
+ onClose={() => setPopupMode(null)}
+ />
<PageTemplatePopup
- open={openPopup === "new"}
- pageTemplate={currentSponsorPage}
- onClose={() => setOpenPopup(null)}
+ open={popupMode === "new" || popupMode === "edit"}
+ pageTemplate={popupMode === "edit" ? currentSponsorPage : null}
+ onClose={() => setPopupMode(null)}
onSave={handleSaveSponsorPage}
/>Also applies to: 232-237
🤖 Prompt for AI Agents
In `@src/pages/sponsors/sponsor-pages-list-page/index.js` around lines 75 - 78,
The popup uses the same "new" state and shared currentSponsorPage for both
creating and editing, so opening "new" after an edit can reuse the existing id
and overwrite; update the flow so edit and create use distinct modes (e.g.,
setOpenPopup("edit") in handleRowEdit and setOpenPopup("new") only for creation)
or explicitly clear/reset currentSponsorPage when opening the "new" popup;
change handleRowEdit to set an "edit" mode (or set a separate isEditing flag)
and ensure any code that opens the create popup (the other place where
setOpenPopup("new") is called) resets currentSponsorPage/template to an empty
object so new saves create instead of update.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ref: https://app.clickup.com/t/86b79919e
Signed-off-by: Tomás Castillo tcastilloboireau@gmail.com
Summary by CodeRabbit