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

Feature/jc 726 authorization implementation #11

Open
wants to merge 36 commits into
base: feature/JC-702-initialize-anime-app
Choose a base branch
from
Open

Feature/jc 726 authorization implementation #11

wants to merge 36 commits into from

Conversation

TebyakinaEkaterina
Copy link
Contributor

Implement registration and authorization

@TebyakinaEkaterina
Install packages for forms JC-726
97f9bb3
@TebyakinaEkaterina
Add registration form JC-726
1867987
@TebyakinaEkaterina
Add registration page JC-726
4f39880
@TebyakinaEkaterina
Setup routes for registration page JC-726
219e8c6
@TebyakinaEkaterina
Add authorization service JC-726
2e6dbe4
@TebyakinaEkaterina
Add local storage service JC-726
8552ce4
@TebyakinaEkaterina
Add user service JC-726
4d524a3
@TebyakinaEkaterina
Add dtos for authorization JC-726
d6e2b31
@TebyakinaEkaterina
Add models for authorization JC-726
c150beb
@TebyakinaEkaterina
Add mappers for authorization JC-726
24646a1
@TebyakinaEkaterina
Add api key variable in config JC-726
f674a29
@TebyakinaEkaterina
Add interceptors JC-726
fb12e2c
@TebyakinaEkaterina
Add user slice JC-726
edb7893
@TebyakinaEkaterina
Add registration components JC-726
fceb695
@TebyakinaEkaterina
Add login components JC-726
8ca6344
@TebyakinaEkaterina
Setup routes for login page Jc-726
e2c1536
@TebyakinaEkaterina
Add authorization menu JC-726
009e660
@TebyakinaEkaterina
Add display server error for login form JC-726
2cfd356
@TebyakinaEkaterina
Implement server errors handling JC-726
3b29277
@TebyakinaEkaterina
Fix access token refreshing JC-726
bc0c742
@TebyakinaEkaterina
Add comment to get user method JC-726
1f17128
@TebyakinaEkaterina
Fix JSDoc comments JC-726
2140a60
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 23, 2024 •
edited
Loading

PR Preview Action v1.4.7
🚀 Deployed preview to https://saritasa-nest.github.io/camp-js-2024-ekaterina-tebyakina/pr-preview/pr-11/
on branch gh-pages at 2024-09-11 06:28 UTC

@TebyakinaEkaterina TebyakinaEkaterina marked this pull request as draft August 23, 2024 12:49
@TebyakinaEkaterina TebyakinaEkaterina marked this pull request as ready for review August 26, 2024 06:14
artem-sotnichenko-saritasa
artem-sotnichenko-saritasa approved these changes Sep 10, 2024
View reviewed changes
apps/react/src/api/services/localStorageService.ts Outdated Show resolved Hide resolved
apps/react/src/store/user/state.ts Outdated Show resolved Hide resolved
apps/react/src/api/services/authorizationService.ts Outdated Show resolved Hide resolved
apps/react/src/features/login/components/LoginForm/LoginForm.tsx Outdated Show resolved Hide resolved
apps/react/src/features/login/components/LoginForm/LoginForm.tsx Outdated Show resolved Hide resolved
apps/react/src/features/login/components/LoginForm/LoginForm.tsx Outdated Show resolved Hide resolved
apps/react/src/features/login/components/LoginForm/LoginForm.tsx Outdated Show resolved Hide resolved
apps/react/src/utils/handleServerErrors.ts Show resolved Hide resolved
libs/core/mappers/registration-data.mapper.ts Outdated Show resolved Hide resolved
@TebyakinaEkaterina @artem-sotnichenko-saritasa
Update JSDoc comment JC-726
2149e6f 
Co-authored-by: Artem Sotnichenko <98300391+artem-sotnichenko-saritasa@users.noreply.github.com>
@TebyakinaEkaterina @artem-sotnichenko-saritasa
Remove redundant annotation of type JC-726
f40ebb3 
Co-authored-by: Artem Sotnichenko <98300391+artem-sotnichenko-saritasa@users.noreply.github.com>
@TebyakinaEkaterina @artem-sotnichenko-saritasa
Remove redundant annotation of type JC-726
faa2dff 
Co-authored-by: Artem Sotnichenko <98300391+artem-sotnichenko-saritasa@users.noreply.github.com>
@TebyakinaEkaterina @artem-sotnichenko-saritasa
Change the approach to converting a value to boolean JC-726
773c72e 
Co-authored-by: Artem Sotnichenko <98300391+artem-sotnichenko-saritasa@users.noreply.github.com>
@TebyakinaEkaterina
Extend eslint config so this rule is ignored in *.mapper.ts files JC-726
8792b91
@TebyakinaEkaterina
Fix types annotations JC-726
cffdc1b
@TebyakinaEkaterina
Implement a service for urls JC-726
861791c
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Sep 11, 2024

Quality Gate Passed Quality Gate passed

Issues
8 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

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

@artem-sotnichenko-saritasa artem-sotnichenko-saritasa artem-sotnichenko-saritasa approved these changes

@DaniilMaslovSaritasa DaniilMaslovSaritasa Awaiting requested review from DaniilMaslovSaritasa DaniilMaslovSaritasa was automatically assigned from saritasa-nest/saritasa-frontend

@viktor-sakhno-saritasa viktor-sakhno-saritasa Awaiting requested review from viktor-sakhno-saritasa viktor-sakhno-saritasa was automatically assigned from saritasa-nest/saritasa-frontend

@andrey-gazzaev andrey-gazzaev Awaiting requested review from andrey-gazzaev andrey-gazzaev was automatically assigned from saritasa-nest/saritasa-frontend

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TebyakinaEkaterina @artem-sotnichenko-saritasa