-
Notifications
You must be signed in to change notification settings - Fork 939
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Server Rendering (previously Client): ActionClass & AttributeClass Overviews #495
Server Rendering (previously Client): ActionClass & AttributeClass Overviews #495
Conversation
@ShubhamPalriwala is attempting to deploy a commit to the formbricks Team on Vercel. A member of the Team first needs to authorize it. |
81bea12
to
2699608
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ShubhamPalriwala 💪 Looks great so far. I left a few comments 😊
apps/web/app/environments/[environmentId]/[actionsAndAttributesOption]/layout.tsx
Outdated
Show resolved
Hide resolved
apps/web/app/environments/[environmentId]/[actionsAndAttributesOption]/page.tsx
Outdated
Show resolved
Hide resolved
...p/environments/[environmentId]/[actionsAndAttributesOption]/(attributes)/AttributeLoader.tsx
Outdated
Show resolved
Hide resolved
apps/web/components/actions_attributes/ActionsAttributesTabs.tsx
Outdated
Show resolved
Hide resolved
@ShubhamPalriwala pls follow up :) |
abc283b
to
599f26c
Compare
apps/web/app/environments/[environmentId]/(actionsAndAttributes)/actions/ActionClassesTable.tsx
Outdated
Show resolved
Hide resolved
apps/web/app/environments/[environmentId]/(actionsAndAttributes)/actions/ActionRowData.tsx
Outdated
Show resolved
Hide resolved
.../app/environments/[environmentId]/(actionsAndAttributes)/attributes/AttributeSettingsTab.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes 🤗 I fixed some small issues but also left a few comments. Would be great if you can update these last things 🤗
apps/web/app/environments/[environmentId]/(actionsAndAttributes)/actions/ActionSettingsTab.tsx
Outdated
Show resolved
Hide resolved
apps/web/app/environments/[environmentId]/(actionsAndAttributes)/actions/ActionSettingsTab.tsx
Outdated
Show resolved
Hide resolved
...web/app/environments/[environmentId]/(actionsAndAttributes)/actions/AddNoCodeActionModal.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job 🎉🤗
…#495) * feat: server rendering of event actions summary page & server actions * chore: renaming event to action and minor refactoring * fix: logging message * delete: unnecessary file * feat: migrate attributes overview page * feat: impl grouped page & layout, logically differentiate attributes and actions * pnpm format * fix: logical addressing of dirs and minot bugs * move: actionsAndAttributes navbar to dedicated dir from components * fix: use server-only build-time checks and move actionsAttributes navbar * revert: unnecessary docker compose changes * resolve merge conflicts dynamically * fix: address feedback comments * use sparkles icon from heroicons * fix updated action not updating in table * remove async from client function due to warning * move router.refresh in AddNoActionModal * small rename * feat: replace swr w server action in ActionSettingsTab * replace custom error with ResourceNotFoundError error class --------- Co-authored-by: Matthias Nannt <mail@matthiasnannt.com>
…#495) * feat: server rendering of event actions summary page & server actions * chore: renaming event to action and minor refactoring * fix: logging message * delete: unnecessary file * feat: migrate attributes overview page * feat: impl grouped page & layout, logically differentiate attributes and actions * pnpm format * fix: logical addressing of dirs and minot bugs * move: actionsAndAttributes navbar to dedicated dir from components * fix: use server-only build-time checks and move actionsAttributes navbar * revert: unnecessary docker compose changes * resolve merge conflicts dynamically * fix: address feedback comments * use sparkles icon from heroicons * fix updated action not updating in table * remove async from client function due to warning * move router.refresh in AddNoActionModal * small rename * feat: replace swr w server action in ActionSettingsTab * replace custom error with ResourceNotFoundError error class --------- Co-authored-by: Matthias Nannt <mail@matthiasnannt.com>
…#495) * feat: server rendering of event actions summary page & server actions * chore: renaming event to action and minor refactoring * fix: logging message * delete: unnecessary file * feat: migrate attributes overview page * feat: impl grouped page & layout, logically differentiate attributes and actions * pnpm format * fix: logical addressing of dirs and minot bugs * move: actionsAndAttributes navbar to dedicated dir from components * fix: use server-only build-time checks and move actionsAttributes navbar * revert: unnecessary docker compose changes * resolve merge conflicts dynamically * fix: address feedback comments * use sparkles icon from heroicons * fix updated action not updating in table * remove async from client function due to warning * move router.refresh in AddNoActionModal * small rename * feat: replace swr w server action in ActionSettingsTab * replace custom error with ResourceNotFoundError error class --------- Co-authored-by: Matthias Nannt <mail@matthiasnannt.com>
What does this PR do?
/environments/[environmentId]/[actionsAndAttributesOption]
is now a param based on which we either load the actions page or the attributes pageType of change
Checklist
pnpm build
console.logs
git pull origin main