-
Notifications
You must be signed in to change notification settings - Fork 174
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
feat: add cascade delete for prompt #84
Merged
120axuz
merged 2 commits into
feature/org-members-invitations
from
feature/delete-prompt
Jun 17, 2023
Merged
feat: add cascade delete for prompt #84
120axuz
merged 2 commits into
feature/org-members-invitations
from
feature/delete-prompt
Jun 17, 2023
Conversation
This file contains 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
arielweinberger
requested changes
Jun 15, 2023
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.
Looks great overall, just a few minor changes.
ItayElgazar
reviewed
Jun 15, 2023
apps/console/src/app/components/prompts/DeletePromptConfirmationModal.tsx
Outdated
Show resolved
Hide resolved
120axuz
force-pushed
the
feature/delete-prompt
branch
from
June 17, 2023 16:00
21e0a58
to
52897e8
Compare
arielweinberger
approved these changes
Jun 17, 2023
arielweinberger
added a commit
that referenced
this pull request
Jun 17, 2023
* feat: invitation creation, listing, deletion and acceptance * refactor: breadcrumbs * implement orgs wrapper * fetch projects by organization ID * cleanup * org member + invitations management * implement org settings + rename * reset settings after save * permissions by org rather than project * remove project memberships * add kafka and org invitation sent event * make kafka module more robust * fix formatting * feat: hide org settings page from member users * feat: move api keys to organization level * feat: use role dropdown to change invited member role * org invitations * gitmodules * set font-weight of headings to 400 * fix imports * fix formatting * fix kafka offline schema generation * add scripts folder to tsconfig * remove redundant function * fix kafka producer connect * chore: code tweaks & folder restructure (#87) * chore: code tweaks & folder restructure * delete docker * chore: add more logs * chore: add more logs * chore: remove redandunt if * chore: change orgs * change logs * fix: build * fix: fix issue with wrong org fetched from local storage (#88) * fix: handle errors * feat: add cascade delete for prompt (#84) * feat: add cascade delete for prompt * fix: linting * test dispatch * test dispatch * test dispatch * test dispatch with payload * test with submodule * sync submodule * rename CI event * update submodule * update submodule version * update pezzo-private submodule * update submodule * update submodule * add docker metadata * update submodule * delete entrypoint, update submodule * remove entrypoint * update submodule * update submodules * update submodule * add short sha * remove short sha * update ci and submodules * update submodule * emit shortSha in event * fix short sha emission * fix short sha * make short sha * update varaibles * fix formatting --------- Co-authored-by: 120axuz <ran@pezzo.ai> Co-authored-by: Itay Elgazar <Itayekk1@gmail.com> Co-authored-by: 120axuz <136251517+120axuz@users.noreply.github.com>
ItayElgazar
added a commit
that referenced
this pull request
Jun 21, 2023
* feat: invitation creation, listing, deletion and acceptance * refactor: breadcrumbs * implement orgs wrapper * fetch projects by organization ID * cleanup * org member + invitations management * implement org settings + rename * reset settings after save * permissions by org rather than project * remove project memberships * add kafka and org invitation sent event * make kafka module more robust * fix formatting * feat: hide org settings page from member users * feat: move api keys to organization level * feat: use role dropdown to change invited member role * org invitations * gitmodules * set font-weight of headings to 400 * fix imports * fix formatting * fix kafka offline schema generation * add scripts folder to tsconfig * remove redundant function * fix kafka producer connect * chore: code tweaks & folder restructure (#87) * chore: code tweaks & folder restructure * delete docker * chore: add more logs * chore: add more logs * chore: remove redandunt if * chore: change orgs * change logs * fix: build * fix: fix issue with wrong org fetched from local storage (#88) * fix: handle errors * feat: add cascade delete for prompt (#84) * feat: add cascade delete for prompt * fix: linting * test dispatch * test dispatch * test dispatch * feat: type safety through kafka lib (#89) * feat: ci dispatch event (#91) * test dispatch with payload * test with submodule * sync submodule * rename CI event * update submodule * update submodule version * update pezzo-private submodule * update submodule * update submodule * add docker metadata * update submodule * delete entrypoint, update submodule * remove entrypoint * update submodule * update submodules * update submodule * add short sha * remove short sha * update ci and submodules * update submodule * emit shortSha in event * fix short sha emission * fix short sha * make short sha * update varaibles * fix formatting * enable source maps * clearner way to build with sourcemaps * test build libs from source * add ^build dependency to console * add reflect-metadata * feat: logout page * cleanup localstorage org id when signing out * use Pezzo ID rather than supertokens ID * use plural private member for service * update submodule * fix: formatting * fix: dependencies * update submodule * feat: add supertokensUserId to RequestUser interface * fix: formatting * temporarily allow circular dependency for @pezzo/kafka * fix formatting * feat: new UI look (#94) * feat: new Pezzo brand look * sync submodules * feat(readme): new look * update image * reduce logo size on readme * reimplement org selector * fix: hide create project for non admins (#93) * fix: hide create project buttons for non admin users * fix: import * fix: admin org * remove org admin import * fix: remove pascal case from prompt name * fix: remove role & prevent from invite yourself * fix: wrong if statement * chore: add gpt-3.5-turbo-16k * feat: add new openai model settings * fix: update openai's cost * fix: ai21 labs schema & change initial editor value to be text * chore: update format * chore: bump versions --------- Co-authored-by: Ariel Weinberger <weinberger.ariel@gmail.com> Co-authored-by: 120axuz <136251517+120axuz@users.noreply.github.com>
ItayElgazar
pushed a commit
that referenced
this pull request
Jun 28, 2023
* feat: add cascade delete for prompt
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.
This PR adds the ability to delete a prompt. The deletion action cascades and removes all other entities that are related and tied to the prompt entity.
The new delete button can be found in the settings tab (new)