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

Release Sharing and Storage to PROD #1387

Merged
merged 114 commits into from
Aug 2, 2021
Merged

Release Sharing and Storage to PROD #1387

merged 114 commits into from
Aug 2, 2021

Conversation

FSM1
Copy link
Contributor

@FSM1 FSM1 commented Aug 2, 2021

No description provided.

RyRy79261 and others added 30 commits June 15, 2021 15:55
* Cleared warnings

* Updated names

* Essentially done

* remove hardcoded pin

* api-spec update and fixed

* add status column and show all pins

* rc3 everywhere

Co-authored-by: Thibaut Sardan <github@thib.top>
Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
* ui ready

* adding username functions

* wip

* wip

* chenging to form

* profile update

* adding username reeady

* lingui extract

* disable email

* removed FB logo

* changes and comments

* profile update

* Update packages/common-components/src/TextInput/FormikTextInput.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/common-components/src/TextInput/TextInput.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Components/Modules/Settings/Profile.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Contexts/UserContext.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* lingui extract

* Update packages/files-ui/src/Components/Modules/Settings/Profile.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Contexts/UserContext.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Contexts/UserContext.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Components/Modules/Settings/Profile.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Contexts/UserContext.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* suggested changes

* lingui extract

* pushed to 300ms debounce

* lingui extract

Co-authored-by: GitHub Actions <actions@github.com>
Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
* readme edit and issue template

* feature request

* move logo

* logo centered

* README.md
* fix broken things

* rewire for new apis

* lingui extract

* clean up linting errors

* Align paths and test

* post merge fixes

* Apply suggestions from code review

Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>

* Update packages/files-ui/src/Utils/pathUtils.ts

Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>

* update context and hook names

* fix search

* fix bin folder navigation

* fix grid item navigation

* init

* add permission to buckets

* Nav and filtering

* lingui extract

* with some todos

* lint

* basics

* lingui extract

* User circles

* add storage bucket list and bucket details

* clean up file browser modals

* fix linting

* fix CRLF

* wire up file upload

* clean up styling

* refresh buckets on delete

* remove some styling

* remove bulk ops

* user icons with tooltip

* loader

* lingui extract

* nav color

* Revert "nav color"

This reverts commit 9666ba4.

* Revert "loader"

This reverts commit 3b0c058.

* lingui extract

* Revert "Merge branch 'feat/tbaut-shared-overview-1041' of github.com:ChainSafe/files-ui into feat/storage-buckets-overview-1104"

This reverts commit 6c8a65a, reversing
changes made to bdf5fdc.

* Apply suggestions from code review

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>

* implement feedback

* more cleaning up

* clean up unused imports

* show the failed pins

* add pinning bucket to size calc

* Hide facebook login

* remove title

* fix broken import and remove unused icon

* fix app name

* rename to Storage

Co-authored-by: GitHub Actions <actions@github.com>
Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>
Co-authored-by: Thibaut Sardan <github@thib.top>
Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
* fix broken things

* rewire for new apis

* lingui extract

* clean up linting errors

* Align paths and test

* post merge fixes

* Apply suggestions from code review

Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>

* Update packages/files-ui/src/Utils/pathUtils.ts

Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>

* update context and hook names

* fix search

* fix bin folder navigation

* fix grid item navigation

* init

* add permission to buckets

* Nav and filtering

* lingui extract

* with some todos

* lint

* basics

* lingui extract

* User circles

* user icons with tooltip

* loader

* nav color

* lingui extract

* please codacy

* overflow visible

Co-authored-by: Michael Yankelev <myankelev@gmail.com>
Co-authored-by: GitHub Actions <actions@github.com>
Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>
* with pipe

* quote

* apply to all

* Update packages/files-ui/cypress/tests/file-management-spec.ts

Co-authored-by: Ryan Noble <ryanjnoble@gmail.com>

Co-authored-by: Ryan Noble <ryanjnoble@gmail.com>
* Update commands.ts to use page object properties

* Add missing newline

* Fix mistake by adding missing click command
* initial key management ui

* clean up create call

* Show-Hide API Secret

* fix api helper

* revert web3 sign message

* Update packages/storage-ui/src/Components/Modules/ApiKeys.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* resolve some feedback

* use callbacks

* set correct data-id

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>
* prevent dnd when editing

* remove stray test
* disabled button

* Update packages/files-ui/src/Components/Modules/Settings/Profile.tsx

Co-authored-by: Ryan Noble <ryanjnoble@gmail.com>

Co-authored-by: Ryan Noble <ryanjnoble@gmail.com>
* fix lookup validation

* nits

* lingui extract

* Apply suggestions from code review

Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>

* lingui extract

* trans

Co-authored-by: GitHub Actions <actions@github.com>
Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>
* add ternary autofix and unused vars

* suggestions
* fix overflow

* some more

* storage rather than Files
* no and de

* yarn extract

Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
* mobile

* fix elipsis and show shared with

Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
* fix alerts

* fix regex

* indentation
* set the base

* run only on specific file changes

* yml indentation

* nit

* Update .github/workflows/test-storage.yml

* wait more for the first button to appear

* better page, back to 120 and video

* if failure() video

* arg

* remove test file

* gitignore again

* give me a video please

* back to electron who knows..

* wait 20min..

* Try headless test execution in ci

* back to 2min

* different caches?

* Try headless chrome execution in ci

* Revert headless execution change, keep chrome as browser

* Try replacing localhost with ip to troubleshoot ci issues

* Try accessing app in ci with https

* Revert changes after unsuccessful testing

* Try using special docker dns name for accessing localhost

* Try a different way of executing tests in ci

* Try alternative command to start tests in ci

* Revert previous changes after unsuccessful testing

* Add react app parameter to env

* Try a different docker container for cypress tests

* Revert back to latest container after testing

* Add baseUrl to cypress.json

* Remove baseUrl from cypress json after it made no difference

* Prevent browser automatically launching after server start

* Remove env variables that made no difference

* Try a different hosted runner

* Try earliest supported version of ubuntu as host runner

* Try macOS based hosted runner

* Revert host runner after unsuccessful testing

* Try waiting and reloading before attempting first click

* Try waiting and reloading before attempting first click in storage tests

* Remove temp changes after testing

* Try temp amendment of post install script

* remove compile and add it to the start

Co-authored-by: Andrew Snaith <andrew@chainsafe.io>
* Started

* Updated with refactors

* Skeletoned

* lingui extract

* Correct api package up

* Access logic & ID extraction done

* Bucket extracting ready

* Fixing recover

* Trying to fix the routing

* Tried reordering

* Fix routing

* Fixing small items

* Fixed crash issue

* Uploading done

* Merging dev

* Fixed routing

* lingui extract

* Access control done

* DnD access control done

* Owner spacing fixed

* REmoved toto

* Pruned

* lingui extract

* lingui extract

* Pulling permission object

* Fixed swith error

* Minor changes

* Updated route url

* fix build

* remove survey and show bucket name

Co-authored-by: GitHub Actions <actions@github.com>
Co-authored-by: Michael Yankelev <myankelev@gmail.com>
Co-authored-by: Thibaut Sardan <github@thib.top>
* refactor

* stray comment
* wire up create share in context

* wire up create share modal

* make eslint :)

* add translations

* lingui extract

* fix callback

* fix buttons container

* remove unused import

* make eslint :D

* hide dropdown and separator

* Apply suggestions from code review

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* lingui extract

* resolve feedback

* lingui extract

* rename type and fix regex

* lowercase address

* rename file to match component

* make sure sharedFolderUsers is an array before mapping

* Apply suggestions from code review

Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>
Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* update styling

* prevent owner from adding himself

* revert to labels

* resolve duplicate key

* add validation to form

* clear form contents

* make LGTM :D

Co-authored-by: GitHub Actions <actions@github.com>
Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
Co-authored-by: Tanmoy Basak Anjan <tanmoy3399@gmail.com>
* support file and folder deletion

* nit

Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
Currently translated at 90.8% (218 of 240 strings)

Translation: ChainSafe Files/Chainsafe Files user interface
Translate-URL: https://hosted.weblate.org/projects/chainsafe-files/chainsafe-files-user-interface/de/

Co-authored-by: J. Lavoie <j.lavoie@net-c.ca>
* wire up create share in context

* reshuffled

* rename ready before API

* using shared page

* rename with prev file structure

* removed unnecessary code

* added loading

* wire up create share modal

* make eslint :)

* add translations

* lingui extract

* fix callback

* fix buttons container

* remove unused import

* Update packages/files-ui/src/Contexts/FilesContext.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* renamed isEditing

* rename state fix

* make eslint :D

* hide dropdown and separator

* Apply suggestions from code review

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* lingui extract

* resolve feedback

* lingui extract

* rename type and fix regex

* lowercase address

* rename file to match component

* make sure sharedFolderUsers is an array before mapping

* removed delete

* lingui extract

* Use BucketKeyPermission instead of missleading Bucket (#1221)

* use BucketKeyPermission instead of missleading Bucket

* avoid props drilling

* Revert "avoid props drilling"

This reverts commit 3817c91.

* permission check

Co-authored-by: tanmoy12 <tanmoy3399@gmail.com>

Co-authored-by: Michael Yankelev <myankelev@gmail.com>
Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
Co-authored-by: GitHub Actions <actions@github.com>
Tbaut and others added 18 commits July 28, 2021 13:08
* remove button and useFormik everywhere

* make sure the name is reset on esc and add a test for it

* fix test

Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
* Translated using Weblate (German)

Currently translated at 90.8% (218 of 240 strings)

Translation: ChainSafe Files/Chainsafe Files user interface
Translate-URL: https://hosted.weblate.org/projects/chainsafe-files/chainsafe-files-user-interface/de/

* Translated using Weblate (French)

Currently translated at 100.0% (248 of 248 strings)

Translation: ChainSafe Files/Chainsafe Files user interface
Translate-URL: https://hosted.weblate.org/projects/chainsafe-files/chainsafe-files-user-interface/fr/

* Translated using Weblate (French)

Currently translated at 99.6% (258 of 259 strings)

Translation: ChainSafe Files/Chainsafe Files user interface
Translate-URL: https://hosted.weblate.org/projects/chainsafe-files/chainsafe-files-user-interface/fr/

* Translated using Weblate (French)

Currently translated at 99.6% (258 of 259 strings)

Translation: ChainSafe Files/Chainsafe Files user interface
Translate-URL: https://hosted.weblate.org/projects/chainsafe-files/chainsafe-files-user-interface/fr/

Co-authored-by: J. Lavoie <j.lavoie@net-c.ca>
Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
* update label and icon

* lingui extract

* fix unused variable

Co-authored-by: GitHub Actions <actions@github.com>
Co-authored-by: Thibaut Sardan <github@thib.top>
* always allow recovery from the bin

* lingui extract

* lingui extract

Co-authored-by: GitHub Actions <actions@github.com>
)

* fix

* lingui extract

Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
Co-authored-by: GitHub Actions <actions@github.com>
* Added filtering for filesystem type

* Pull from context

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
* Added overrides for react select

* Standardised

* Refactored select component

* Resolved the background color

* lingui extract

* Fixed backgrounds

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
Co-authored-by: GitHub Actions <actions@github.com>
Co-authored-by: Andrew Snaith <asnaith@users.noreply.github.com>
* remove duplicate

* remove svg
* Exposed summary request

* Wired into more functions

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
* remove no verify to see

* lingui extract

* with ssh key

* translation change

* fix workflow

* trigger extract

* move key up

* lingui extract

* apply to all lingui extract

Co-authored-by: GitHub Actions <actions@github.com>
* merged

* spacings

* linings

* renames

* mode typee

* error handling update

* lingui extract

* lingui extract

* merging ready

* lingui extract

* renames

* Update packages/files-ui/src/Components/Modules/FileBrowsers/hooks/useLookupUser.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Components/Modules/FileBrowsers/hooks/useLookupUser.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Components/Modules/FileBrowsers/hooks/useLookupUser.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Components/Modules/FileBrowsers/CreateOrEditSharedFolderModal.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* Update packages/files-ui/src/Components/Modules/FileBrowsers/CreateOrEditSharedFolderModal.tsx

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>

* duplicates

* removed duplicates

* meerge new users

* lingui extract

* meerged

Co-authored-by: Michael Yankelev <12774278+FSM1@users.noreply.github.com>
Co-authored-by: GitHub Actions <actions@github.com>
Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
* Translated using Weblate (Norwegian Bokmål)

Currently translated at 44.9% (128 of 285 strings)

Translation: ChainSafe Files/Chainsafe Files user interface
Translate-URL: https://hosted.weblate.org/projects/chainsafe-files/chainsafe-files-user-interface/nb_NO/

* Translated using Weblate (French)

Currently translated at 99.6% (285 of 286 strings)

Translation: ChainSafe Files/Chainsafe Files user interface
Translate-URL: https://hosted.weblate.org/projects/chainsafe-files/chainsafe-files-user-interface/fr/

Co-authored-by: Allan Nordhøy <epost@anotheragency.no>
Co-authored-by: J. Lavoie <j.lavoie@net-c.ca>
Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
* update bnc-onboard to 1.32.0

* lingui extract

Co-authored-by: Thibaut Sardan <33178835+Tbaut@users.noreply.github.com>
Co-authored-by: GitHub Actions <actions@github.com>
* upgarde to v8.1

* no pipe in storage

* lingui extract

Co-authored-by: GitHub Actions <actions@github.com>
@render
Copy link

render bot commented Aug 2, 2021

@CLAassistant
Copy link

CLAassistant commented Aug 2, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
4 out of 5 committers have signed the CLA.

✅ FSM1
✅ tanmoyAtb
✅ RyRy79261
✅ Tbaut
❌ weblate
You have signed the CLA already but the status is still pending? Let us recheck it.

@lgtm-com
Copy link

lgtm-com bot commented Aug 2, 2021

This pull request introduces 12 alerts and fixes 16 when merging 5553caf into ae88827 - view on LGTM.com

new alerts:

  • 8 for Duplicate character in character class
  • 4 for Unused variable, import, function or class

fixed alerts:

  • 8 for Duplicate character in character class
  • 4 for Identical operands
  • 2 for Useless conditional
  • 1 for Useless assignment to local variable
  • 1 for Unused variable, import, function or class

@render
Copy link

render bot commented Aug 2, 2021

Your Render PR Server at https://files-ui-prod-pr-1387.onrender.com is now live!

View it on your dashboard at https://dashboard.render.com/static/srv-c43vsrpg7hp3ob1bqtn0.

Copy link
Collaborator

@Tbaut Tbaut left a comment

Choose a reason for hiding this comment

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

it builds I think :)

@FSM1 FSM1 merged commit fa99c60 into prod Aug 2, 2021
@lgtm-com
Copy link

lgtm-com bot commented Aug 2, 2021

This pull request fixes 24 alerts when merging 5a17e17 into ae88827 - view on LGTM.com

fixed alerts:

  • 16 for Duplicate character in character class
  • 4 for Identical operands
  • 2 for Useless conditional
  • 1 for Useless assignment to local variable
  • 1 for Unused variable, import, function or class

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants