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.

Sign up for GitHub

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

Jump to bottom

Review (not for merge) #10

Open
wants to merge 137 commits into
base: collaborators-review
Choose a base branch
from
Open

Review (not for merge) #10

wants to merge 137 commits into from

Conversation

MaksGovor
Copy link
Member

No description provided.

Uryn Dmytro and others added 30 commits October 21, 2021 10:29
added package*.json files
cfc1f82
-m removed unnecessary react code and files
13466b5
add dot files
1d72878
changed dot files
c9b7e02
configured react
d3b3cf7
add code of conduct and contributing
b722652
improved package.json and added yarn.lock
d63591c
@MaksGovor
Init project env
fc347d1
@Honchar007
implement login page
022d326
@Honchar007
Merge pull request #1 from Intelligent-Cloud-System/feature/register-…
c1c472b 
…page
@Honchar007
implement register page and validation
f9ea037
@Honchar007
Merge pull request #2 from Intelligent-Cloud-System/feature/register-…
41921c3 
…page

implement register page and validation
@tedi4t
add swagger generate
4a5354e
@tedi4t
generate swagger clients
618822c
@tedi4t
auth integration
eefd6e7
@tedi4t
small style changes
9175182
@tedi4t
injectFirst
dd01ca6
@MaksGovor
Merge pull request #3 from Intelligent-Cloud-System/feature/auth-inte…
2e8ac45 
…gration

auth integration
@MaksGovor
Add husky pre-commit
04925df
@MaksGovor
Merge branch 'dev' into feature/add-husky
93c8db1
@tedi4t
add auto token refresh
e17941c
@tedi4t
Merge pull request #4 from Intelligent-Cloud-System/feature/add-husky
79a887f 
Feature/add-husky
@tedi4t
generate-clients
f0a5ef4
@MaksGovor
Merge pull request #5 from Intelligent-Cloud-System/feature/auth-inte…
ac67f3a 
…gration

add auto token refresh
@MaksGovor
Add favicon & logo
5de1228
@MaksGovor
Add base layout
c9aa5ef
@Honchar007
feature\implement-upload-ui
9401016
@tedi4t
add notification hook for errors
5f95f57
@MaksGovor
Change nav structure
195b6d8
@tedi4t
small changes
06c5778
MaksGovor and others added 13 commits May 26, 2022 23:43
@MaksGovor
Fix: fix code formatting
f65c3f5
@MaksGovor
Fix: fix repeating code
0a50602
@MaksGovor
Fix: small fixes
a23eede
@MaksGovor
Fix: fix styles logic
be2fe32
@MaksGovor
Feat: use addaptive scroll bar in files container
5bbe697
@MaksGovor
Feat: update clients
ee458cc
@MaksGovor @tedi4t
Fix: using default import
d89243b 
Co-authored-by: Uryn Dmytro <[email protected]>
@MaksGovor @tedi4t
Rename downloadFilesByLink to downloadFilesFromSignedUrls
761f908 
Co-authored-by: Uryn Dmytro <[email protected]>
@MaksGovor @tedi4t
Fix: rename existingContent to content
2274c49 
Co-authored-by: Uryn Dmytro <[email protected]>
@MaksGovor
Fix: use onError method for Error handling
1d565f3
@MaksGovor
Merge pull request #19 from Intelligent-Cloud-System/feature/file-fol…
af3d74a 
…der-pretty-view

Feature: file folder pretty view
@MaksGovor
Merge pull request #20 from Intelligent-Cloud-System/dev
fc3755d 
Merge dev in production
add .idea to gitignore
45c4b24
tshemsedinov
tshemsedinov approved these changes Jun 9, 2022
View reviewed changes
Copy link

@tshemsedinov tshemsedinov left a comment

Choose a reason for hiding this comment

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

  • Use fs.promises
  • I'd like to see websockets instead of http
  • Try to optimize dependencies, for example node-fetch and so on

@tshemsedinov
Copy link

photo_2022-05-19_21-35-45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nosovk nosovk nosovk left review comments

@tshemsedinov tshemsedinov tshemsedinov approved these changes

Assignees

@tedi4t tedi4t

@MaksGovor MaksGovor

@YAGoOaR YAGoOaR

Labels
code review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@MaksGovor @tshemsedinov @nosovk @tedi4t @YAGoOaR @Honchar007