Skip to content
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

fix: prompt template not being saved on advanced modal #2488

Merged
merged 14 commits into from
Jul 3, 2024

Conversation

lucaseduoli
Copy link
Collaborator

  • Fixed prompt template changing by the Advanced modal not triggering changes
  • Fixed SetDB attribution on Advanced modal
  • Removed Save Changes from Advanced modal. Changes are now triggered automatically.

@lucaseduoli lucaseduoli linked an issue Jul 2, 2024 that may be closed by this pull request
@lucaseduoli lucaseduoli self-assigned this Jul 2, 2024
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. bug Something isn't working javascript Pull requests that update Javascript code labels Jul 2, 2024
@lucaseduoli
Copy link
Collaborator Author

Waiting for @Cristhianzl to fix Frontend tests

@lucaseduoli lucaseduoli requested review from Cristhianzl and removed request for ogabrielluiz July 2, 2024 18:57
Copy link
Contributor

@anovazzi1 anovazzi1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 2, 2024
@lucaseduoli lucaseduoli enabled auto-merge (squash) July 3, 2024 17:48
@lucaseduoli lucaseduoli merged commit fe21f90 into main Jul 3, 2024
23 checks passed
@lucaseduoli lucaseduoli deleted the 1561-prompt-template-not-being-saved branch July 3, 2024 17:55
nicoloboschi pushed a commit to datastax/ragstack-ai-langflow that referenced this pull request Jul 10, 2024
)

* Fixed dbvalue on table node cell renderer

* Added Change Advanced hook

* Added Handle New Value hook

* Added Handle Node Class hook

* Added Node Class handler to TableNodeCellRender

* Removed internal state of EditNode, added internal state for NodeClass and made the rows and columns be updated when NodeClass changes

* Added NodeClass as dependencies on useMemo to update columns when NodeClass changes

* Fixed advanced not changing the node

* feat: updating tests without save btn

* Added Close button on editNode

---------

Co-authored-by: cristhianzl <[email protected]>
Co-authored-by: anovazzi1 <[email protected]>
(cherry picked from commit fe21f90)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working javascript Pull requests that update Javascript code lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prompt template not being saved
3 participants