shareable URL modal UI Fixes #1537
Merged
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
tynandebold
reviewed
Sep 22, 2023
tynandebold
added a commit
that referenced
this pull request
Oct 10, 2023
* nodes working Signed-off-by: Rashida Kanchwala <[email protected]> * added pipeline Signed-off-by: Rashida Kanchwala <[email protected]> * fix path Signed-off-by: Rashida Kanchwala <[email protected]> * fix bug and lint Signed-off-by: Rashida Kanchwala <[email protected]> * fix lint Signed-off-by: Rashida Kanchwala <[email protected]> * Add deploy button, modal, and form fields Signed-off-by: Tynan DeBold <[email protected]> * Fix failing tests Signed-off-by: Tynan DeBold <[email protected]> * Fix failing tests Signed-off-by: Tynan DeBold <[email protected]> * Update single file to S3 via file input Signed-off-by: Tynan DeBold <[email protected]> * Update test case Signed-off-by: Tynan DeBold <[email protected]> * Remove package lock changes Signed-off-by: Tynan DeBold <[email protected]> * Hide exp tracking and deploy button when not running Viz locally; remove WS/subscription code Signed-off-by: Tynan DeBold <[email protected]> * Add the beginnings of a /deploy REST endpoint Signed-off-by: Tynan DeBold <[email protected]> * Trying to send credentials to the backend Signed-off-by: Tynan DeBold <[email protected]> * update responses * test * Update UI to remove unneeded fields Signed-off-by: Tynan DeBold <[email protected]> * Fix failing text Signed-off-by: Tynan DeBold <[email protected]> * Workable Prototype * fix rebase * fixing stuff * Successfully returning a 200 from /api/deploy endpoint Signed-off-by: Tynan DeBold <[email protected]> * Remove aws sdk package Signed-off-by: Tynan DeBold <[email protected]> * backend fixes * remove map files when uploading * fix issue on html folder * UI updates for loading, success, error states; resetting modal Signed-off-by: Tynan DeBold <[email protected]> * fix lowerbound on fsspec * refactored to latest * some refactor * fix lint - WIP * refactor work * fix lint * remove old code * further refactor * add error handling and debugging * fix based on review * modify upload static files logic * refactor upload api with latest fsspec * fix unit tests_1 * revert os logic to pathlib * fix static folder issue * fix format and lint errors Signed-off-by: ravi-kumar-pilla <[email protected]> * add unit tests for shareable viz s3deployer Signed-off-by: ravi-kumar-pilla <[email protected]> * add pytests for responses module Signed-off-by: ravi-kumar-pilla <[email protected]> * add s3fs as dependency Signed-off-by: ravi-kumar-pilla <[email protected]> * add temporary no cover for apps Signed-off-by: ravi-kumar-pilla <[email protected]> * update lower reqs * update fsspec * check kedro latest version as 18.0 in e2e tests Signed-off-by: ravi-kumar-pilla <[email protected]> * update fsspec and s3fs requirements to support earliest kedro version Signed-off-by: ravi-kumar-pilla <[email protected]> * add timestamp file for deploy Signed-off-by: ravi-kumar-pilla <[email protected]> * add pytest for timestamp route Signed-off-by: ravi-kumar-pilla <[email protected]> * fix lint and format errors Signed-off-by: ravi-kumar-pilla <[email protected]> * fix server changes and test e2e scenarios Signed-off-by: ravi-kumar-pilla <[email protected]> * try to catch versionConflicterror * Style updates; sending updated args to the backend Signed-off-by: Tynan DeBold <[email protected]> * Re-deploy flow Signed-off-by: Tynan DeBold <[email protected]> * add route /api/project-metadata to provide package version info Signed-off-by: ravi-kumar-pilla <[email protected]> * remove frontend build for backend unit tests Signed-off-by: ravi-kumar-pilla <[email protected]> * remove s3fs requirement to test Signed-off-by: ravi-kumar-pilla <[email protected]> * add s3fs without specific version Signed-off-by: ravi-kumar-pilla <[email protected]> * adjust requirements and add pytests for project metadata Signed-off-by: ravi-kumar-pilla <[email protected]> * test open s3fs requirement Signed-off-by: ravi-kumar-pilla <[email protected]> * test open s3fs requirement Signed-off-by: ravi-kumar-pilla <[email protected]> * test open s3fs requirement Signed-off-by: ravi-kumar-pilla <[email protected]> * Move share viz button to new location; add download icon Signed-off-by: Tynan DeBold <[email protected]> * Update tests Signed-off-by: Tynan DeBold <[email protected]> * Add shareable timestamp component Signed-off-by: Tynan DeBold <[email protected]> * Update timestamp fetch Signed-off-by: Tynan DeBold <[email protected]> * add version info and modify route name from /api/timestamp to /api/deploy-viz-metadata Signed-off-by: ravi-kumar-pilla <[email protected]> * Rename file; add version to timestamp Signed-off-by: Tynan DeBold <[email protected]> * Add back correct isRunningLocally check Signed-off-by: Tynan DeBold <[email protected]> * Remove console.log and update endpoint url Signed-off-by: Tynan DeBold <[email protected]> * add pytests for updated api Signed-off-by: ravi-kumar-pilla <[email protected]> * undo all new requirements * undo all fsspec changes * added s3fs as dependency * fix unit tests * clean up tests * lint * fix lint * fix test and lint and compatibility response * add packaging * packaging reqs * Consume compatibility endpoint; use Dropdown for bucket regions Signed-off-by: Tynan DeBold <[email protected]> * Update failing test Signed-off-by: Tynan DeBold <[email protected]> * Run format-fix Signed-off-by: Tynan DeBold <[email protected]> * Remove unused import Signed-off-by: Tynan DeBold <[email protected]> * fix lint and router link * Update API endpoint Signed-off-by: Tynan DeBold <[email protected]> * Add try/catch; remove console.log Signed-off-by: Tynan DeBold <[email protected]> * Update e2e test Signed-off-by: Tynan DeBold <[email protected]> * Update e2e test, again Signed-off-by: Tynan DeBold <[email protected]> * Drodown component update Signed-off-by: Tynan DeBold <[email protected]> * fix api endpoint * fixes based on reviews * fixes based on reviews * changes based on reviews * fix lint * shareable URL modal UI Fixes (#1537) * shareable url modal UI fix * Build error fix * shareable URL modal form validation fix * fixes based on review * updated cli help definition * update filpath to directory * update filpath to directory * Add the first of the Cypress tests Signed-off-by: Tynan DeBold <[email protected]> * add s3 protocol in the backend * leftover from merge fix * Changing language from deploy to publish Signed-off-by: Tynan DeBold <[email protected]> * Update wording again; return a better error message to the FE; handle the error; move s3 bucket regions to config file Signed-off-by: Tynan DeBold <[email protected]> * Add go back button to error modal stategs Signed-off-by: Tynan DeBold <[email protected]> * Update tests; use fstrings; remove unused CSS Signed-off-by: Tynan DeBold <[email protected]> * E2E cypress test for Shareable URL modal (#1543) * E2E cypress test added for shareable * Renaming from "Deploy" to "Publish" * Code review changes added * update pip fsspec * Update UI with docs links; update feature hits Signed-off-by: Tynan DeBold <[email protected]> * Update feature hints for when Viz is deployed Signed-off-by: Tynan DeBold <[email protected]> * Update release notes Signed-off-by: Tynan DeBold <[email protected]> * Update release notes again Signed-off-by: Tynan DeBold <[email protected]> * Add tracking to create_api_app_from_file Signed-off-by: Tynan DeBold <[email protected]> * fix e2e tests * Revert changes to create_api_app_from_file function Signed-off-by: Tynan DeBold <[email protected]> * Fix for telemetry in sharaeable viz (#1551) * fix telemetry for sharaeable viz Signed-off-by: ravi-kumar-pilla <[email protected]> * fix lint issue Signed-off-by: ravi-kumar-pilla <[email protected]> --------- Signed-off-by: ravi-kumar-pilla <[email protected]> Signed-off-by: Tynan DeBold <[email protected]> Co-authored-by: Tynan DeBold <[email protected]> * Design QA changes Signed-off-by: Tynan DeBold <[email protected]> * Update release file Signed-off-by: Tynan DeBold <[email protected]> * PR review fixes Signed-off-by: Tynan DeBold <[email protected]> * Add back graphql subscription (removal moved to another PR) Signed-off-by: Tynan DeBold <[email protected]> * Update cypress test Signed-off-by: Tynan DeBold <[email protected]> --------- Signed-off-by: Rashida Kanchwala <[email protected]> Signed-off-by: Tynan DeBold <[email protected]> Signed-off-by: ravi-kumar-pilla <[email protected]> Co-authored-by: Rashida Kanchwala <[email protected]> Co-authored-by: Rashida Kanchwala <[email protected]> Co-authored-by: rashidakanchwala <[email protected]> Co-authored-by: ravi-kumar-pilla <[email protected]> Co-authored-by: Jitendra Gundaniya <[email protected]>
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.
Description
Resolves #1, #2 & #3
Development notes
This change also fixing dropdown closing issue where component is used.
Checklist
RELEASE.mdfile