From 930f48bde132657cb2e89a283cf0ec4783898a52 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 4 Oct 2017 20:50:49 +0200 Subject: [PATCH 01/31] Adding Docker installation manual --- Docker installation manual | 113 +++++++++++++++++++++++++++++++++++++ 1 file changed, 113 insertions(+) create mode 100644 Docker installation manual diff --git a/Docker installation manual b/Docker installation manual new file mode 100644 index 00000000..7af9d756 --- /dev/null +++ b/Docker installation manual @@ -0,0 +1,113 @@ +Docker and Docker Compose installation manual + +#Source website: +# https://docs.docker.com/engine/installation/linux/docker-ce/ubuntu/ + +0. Open terminal. + +1. Uninstall old versions of Docker: + + $ sudo apt-get remove docker docker-engine docker.io + +2. Update the apt package index: + + $ sudo apt-get update + +3.Install packages to allow apt to use a repository over HTTPS: + + $ sudo apt-get install \ + apt-transport-https \ + ca-certificates \ + curl \ + software-properties-common + +4. Add Docker’s official GPG key: + + $ curl -fsSL https://download.docker.com/linux/ubuntu/gpg | sudo apt-key add - + + Alternative way(in case of emergency): + + Open following URL in browser(Chrome, if possible): https://download.docker.com/linux/ubuntu/gpg + + $ cat DownloadedGpgFilePath | sudo apt-key add - + +5. Verify that you now have the key using last 8 characters of the fingerprint: + + $ sudo apt-key fingerprint 0EBFCD88 + + You should get sth like this: + + pub 4096R/0EBFCD88 2017-02-22 + Key fingerprint = 9DC8 5822 9FC7 DD38 854A E2D8 8D81 803C 0EBF CD88 + uid Docker Release (CE deb) + sub 4096R/F273FCD8 2017-02-22 + + +6. Use the following command to set up the stable repository: + + $ sudo add-apt-repository \ + "deb [arch=amd64] https://download.docker.com/linux/ubuntu \ + $(lsb_release -cs) \ + stable" + +7. Install Docker CE: + + $ sudo apt-get update + + If any errors occur, force using IPv4 by adding "-o Acquire::ForceIPv4=true update": + + $ sudo apt-get update -o Acquire::ForceIPv4=true update + + + Next step: + + $ sudo apt-get install docker-ce + + As before, if any errors occured, force using IPv4: + + $ sudo apt-get install docker-ce -o Acquire::ForceIPv4=true update + + +8. Verify that Docker CE is installed correctly by running the hello-world image: + + $ sudo docker run hello-world + + +9. If you don’t want to use sudo when you use the docker command, create a Unix group called docker and add users to it: + + Create the docker group: + + $ sudo groupadd docker + + + Add your user to the docker group. + + $ sudo usermod -aG docker $USER + + + Log out and log back in so that your group membership is re-evaluated. + If testing on a virtual machine, it may be necessary to restart the virtual machine for changes to take affect. + + +10. Install Docker Compose + + $ sudo apt install docker-compose + +11.Setup environment: + + From downloaded source code run: + + $ docker-compose up + + and point your browser to [http://localhost:8000](http://localhost:8000) and [http://localhost:4200](http://localhost:4200) + + +12. Enjoy! + + + + + + + + From d34d383efd03ab876d06ee0da7edfb8c167c6d5b Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Fri, 17 Nov 2017 19:25:53 +0100 Subject: [PATCH 02/31] Deleted Docker Manual --- Docker installation manual | 113 ------------------------------------- 1 file changed, 113 deletions(-) delete mode 100644 Docker installation manual diff --git a/Docker installation manual b/Docker installation manual deleted file mode 100644 index 7af9d756..00000000 --- a/Docker installation manual +++ /dev/null @@ -1,113 +0,0 @@ -Docker and Docker Compose installation manual - -#Source website: -# https://docs.docker.com/engine/installation/linux/docker-ce/ubuntu/ - -0. Open terminal. - -1. Uninstall old versions of Docker: - - $ sudo apt-get remove docker docker-engine docker.io - -2. Update the apt package index: - - $ sudo apt-get update - -3.Install packages to allow apt to use a repository over HTTPS: - - $ sudo apt-get install \ - apt-transport-https \ - ca-certificates \ - curl \ - software-properties-common - -4. Add Docker’s official GPG key: - - $ curl -fsSL https://download.docker.com/linux/ubuntu/gpg | sudo apt-key add - - - Alternative way(in case of emergency): - - Open following URL in browser(Chrome, if possible): https://download.docker.com/linux/ubuntu/gpg - - $ cat DownloadedGpgFilePath | sudo apt-key add - - -5. Verify that you now have the key using last 8 characters of the fingerprint: - - $ sudo apt-key fingerprint 0EBFCD88 - - You should get sth like this: - - pub 4096R/0EBFCD88 2017-02-22 - Key fingerprint = 9DC8 5822 9FC7 DD38 854A E2D8 8D81 803C 0EBF CD88 - uid Docker Release (CE deb) - sub 4096R/F273FCD8 2017-02-22 - - -6. Use the following command to set up the stable repository: - - $ sudo add-apt-repository \ - "deb [arch=amd64] https://download.docker.com/linux/ubuntu \ - $(lsb_release -cs) \ - stable" - -7. Install Docker CE: - - $ sudo apt-get update - - If any errors occur, force using IPv4 by adding "-o Acquire::ForceIPv4=true update": - - $ sudo apt-get update -o Acquire::ForceIPv4=true update - - - Next step: - - $ sudo apt-get install docker-ce - - As before, if any errors occured, force using IPv4: - - $ sudo apt-get install docker-ce -o Acquire::ForceIPv4=true update - - -8. Verify that Docker CE is installed correctly by running the hello-world image: - - $ sudo docker run hello-world - - -9. If you don’t want to use sudo when you use the docker command, create a Unix group called docker and add users to it: - - Create the docker group: - - $ sudo groupadd docker - - - Add your user to the docker group. - - $ sudo usermod -aG docker $USER - - - Log out and log back in so that your group membership is re-evaluated. - If testing on a virtual machine, it may be necessary to restart the virtual machine for changes to take affect. - - -10. Install Docker Compose - - $ sudo apt install docker-compose - -11.Setup environment: - - From downloaded source code run: - - $ docker-compose up - - and point your browser to [http://localhost:8000](http://localhost:8000) and [http://localhost:4200](http://localhost:4200) - - -12. Enjoy! - - - - - - - - From c9eb05728f3182209c77dc69c089eedd1c22574e Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 24 Jan 2018 20:03:44 +0100 Subject: [PATCH 03/31] CreatedRegisterComponent --- frontend/src/app/app.module.ts | 6 +++++ .../src/app/register/register.component.css | 0 .../src/app/register/register.component.html | 3 +++ .../app/register/register.component.spec.ts | 25 +++++++++++++++++++ .../src/app/register/register.component.ts | 15 +++++++++++ 5 files changed, 49 insertions(+) create mode 100644 frontend/src/app/register/register.component.css create mode 100644 frontend/src/app/register/register.component.html create mode 100644 frontend/src/app/register/register.component.spec.ts create mode 100644 frontend/src/app/register/register.component.ts diff --git a/frontend/src/app/app.module.ts b/frontend/src/app/app.module.ts index 7f942c8d..bebedc23 100644 --- a/frontend/src/app/app.module.ts +++ b/frontend/src/app/app.module.ts @@ -30,6 +30,7 @@ import { BannerComponent } from './banner/banner.component'; import { FaqOrganizationsComponent } from './static/faq-organizations.component'; import { OfficeComponent } from './static/office/office.component'; import { FaqVolunteersComponent } from './static/faq-volunteers.component'; +import { RegisterComponent } from './register/register.component'; Raven.config(environment.sentryDSN).install(); @@ -68,6 +69,10 @@ const appRoutes: Routes = [ path: 'login', component: LoginComponent, }, + { + path: 'register', + component: RegisterComponent, + }, { path: 'regulations', component: RegulationsComponent @@ -102,6 +107,7 @@ const appRoutes: Routes = [ FaqOrganizationsComponent, OfficeComponent, FaqVolunteersComponent, + RegisterComponent, ], imports: [ BrowserModule, diff --git a/frontend/src/app/register/register.component.css b/frontend/src/app/register/register.component.css new file mode 100644 index 00000000..e69de29b diff --git a/frontend/src/app/register/register.component.html b/frontend/src/app/register/register.component.html new file mode 100644 index 00000000..68065ce1 --- /dev/null +++ b/frontend/src/app/register/register.component.html @@ -0,0 +1,3 @@ +

+ register works!ksjndk skjn +

diff --git a/frontend/src/app/register/register.component.spec.ts b/frontend/src/app/register/register.component.spec.ts new file mode 100644 index 00000000..6c19551b --- /dev/null +++ b/frontend/src/app/register/register.component.spec.ts @@ -0,0 +1,25 @@ +import { async, ComponentFixture, TestBed } from '@angular/core/testing'; + +import { RegisterComponent } from './register.component'; + +describe('RegisterComponent', () => { + let component: RegisterComponent; + let fixture: ComponentFixture; + + beforeEach(async(() => { + TestBed.configureTestingModule({ + declarations: [ RegisterComponent ] + }) + .compileComponents(); + })); + + beforeEach(() => { + fixture = TestBed.createComponent(RegisterComponent); + component = fixture.componentInstance; + fixture.detectChanges(); + }); + + it('should create', () => { + expect(component).toBeTruthy(); + }); +}); diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts new file mode 100644 index 00000000..746778ce --- /dev/null +++ b/frontend/src/app/register/register.component.ts @@ -0,0 +1,15 @@ +import { Component, OnInit } from '@angular/core'; + +@Component({ + selector: 'volontulo-register', + templateUrl: './register.component.html', + styleUrls: ['./register.component.css'] +}) +export class RegisterComponent implements OnInit { + + constructor() { } + + ngOnInit() { + } + +} From f67a3ef6c89f12b4e20e6a9802eec1aae8a1b51e Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 21 Mar 2018 18:02:08 +0100 Subject: [PATCH 04/31] Moved Registration To Angular --- .../apps/volontulo/templates/auth/login.html | 2 +- .../volontulo/templates/common/footer.html | 8 +-- .../volontulo/templates/common/header.html | 2 +- .../templates/emails/registration.html | 2 +- .../templates/emails/registration.txt | 2 +- .../apps/volontulo/templates/homepage.html | 2 +- backend/apps/volontulo/urls.py | 11 +++- backend/apps/volontulo/views/api.py | 63 +++++++++++++++++++ .../account-activation.component.css | 0 .../account-activation.component.html | 4 ++ .../account-activation.component.spec.ts | 25 ++++++++ .../account-activation.component.ts | 39 ++++++++++++ frontend/src/app/app.module.ts | 6 ++ frontend/src/app/auth.models.ts | 10 +++ frontend/src/app/auth.service.ts | 24 +++++++ frontend/src/app/footer/footer.component.html | 6 +- frontend/src/app/header/header.component.html | 2 +- .../src/app/register/register.component.html | 52 ++++++++++++++- .../src/app/register/register.component.ts | 26 +++++++- 19 files changed, 268 insertions(+), 18 deletions(-) create mode 100644 frontend/src/app/account-activation/account-activation.component.css create mode 100644 frontend/src/app/account-activation/account-activation.component.html create mode 100644 frontend/src/app/account-activation/account-activation.component.spec.ts create mode 100644 frontend/src/app/account-activation/account-activation.component.ts diff --git a/backend/apps/volontulo/templates/auth/login.html b/backend/apps/volontulo/templates/auth/login.html index 09dc07cc..63793755 100644 --- a/backend/apps/volontulo/templates/auth/login.html +++ b/backend/apps/volontulo/templates/auth/login.html @@ -5,7 +5,7 @@ {% block content %}

Rejestracja w Volontulo

-
+ {% csrf_token %}
{{ user_form.email.errors }} diff --git a/backend/apps/volontulo/templates/common/footer.html b/backend/apps/volontulo/templates/common/footer.html index 2cc0f357..d3720fe8 100644 --- a/backend/apps/volontulo/templates/common/footer.html +++ b/backend/apps/volontulo/templates/common/footer.html @@ -5,8 +5,8 @@

Nawigacja

Informacje

@@ -14,7 +14,7 @@

Informacje

Strefa organizacji i instytucji

    -
  • Załóż konto
    • +
  • Załóż konto
  • Dodaj nową ofertę
  • Lista organizacji i instytucji
  • FAQ dla organizacji i instytucji
    • @@ -24,7 +24,7 @@

    Strefa organizacji i instytucji

    Strefa wolontariusza

      -
    • Załóż konto
      • +
    • Załóż konto
    • Lista aktualnych ofert
    • Archiwum ofert
    • Lista organizacji i instytucji
      • diff --git a/backend/apps/volontulo/templates/common/header.html b/backend/apps/volontulo/templates/common/header.html index c207232a..90313e21 100644 --- a/backend/apps/volontulo/templates/common/header.html +++ b/backend/apps/volontulo/templates/common/header.html @@ -15,7 +15,7 @@

      {% if user.is_anonymous and not user.is_authenticated %} - Załóż konto + Załóż konto lub Zaloguj się {% else %} diff --git a/backend/apps/volontulo/templates/emails/registration.html b/backend/apps/volontulo/templates/emails/registration.html index 77d4849e..e936f742 100644 --- a/backend/apps/volontulo/templates/emails/registration.html +++ b/backend/apps/volontulo/templates/emails/registration.html @@ -11,7 +11,7 @@ Dziękujemy za rejestrację.

      Zweryfikuj swoje konto na Volontulo wchodząc w poniższy link:
      - Aktywuj + Aktywuj {% endblock %} diff --git a/backend/apps/volontulo/templates/emails/registration.txt b/backend/apps/volontulo/templates/emails/registration.txt index aa40e374..b66390a1 100644 --- a/backend/apps/volontulo/templates/emails/registration.txt +++ b/backend/apps/volontulo/templates/emails/registration.txt @@ -2,5 +2,5 @@ {% block email_content %} Dziękujemy za rejestrację. Zweryfikuj swoje konto na Volontulo wchodząc w przeglądarce pod poniższy link: -{{ protocol }}://{{ domain }}{% url 'activate' uuid %} +{{ angular_root }}/account-activation?activation_token={{ token }} {% endblock %} diff --git a/backend/apps/volontulo/templates/homepage.html b/backend/apps/volontulo/templates/homepage.html index 33de6855..9ef91b03 100644 --- a/backend/apps/volontulo/templates/homepage.html +++ b/backend/apps/volontulo/templates/homepage.html @@ -51,7 +51,7 @@

      {% if user.is_anonymous and not user.is_authenticated %}
      - Załóż konto + Załóż konto
      lub diff --git a/backend/apps/volontulo/urls.py b/backend/apps/volontulo/urls.py index 2360b4e7..584070ea 100644 --- a/backend/apps/volontulo/urls.py +++ b/backend/apps/volontulo/urls.py @@ -44,7 +44,16 @@ api_views.current_user, name='current_user' ), - + url( + r'^api/register', + api_views.register_view, + name='api_register' + ), + url( + r'^api/activate', + api_views.activate, + name='api_activate' + ), # homepage: url(r'^o$', views.homepage, name='homepage'), diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index 4780b1ea..ea776681 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -4,9 +4,14 @@ .. module:: api """ +from django.conf import settings +from django.contrib import messages from django.contrib.auth import authenticate from django.contrib.auth import login from django.contrib.auth import logout +from django.contrib.auth.models import User +from django.db.utils import IntegrityError +from django.shortcuts import redirect from rest_framework import status from rest_framework.decorators import api_view from rest_framework.decorators import authentication_classes @@ -19,6 +24,8 @@ from apps.volontulo import permissions from apps.volontulo import serializers from apps.volontulo.authentication import CsrfExemptSessionAuthentication +from apps.volontulo.lib.email import send_mail +from apps.volontulo.models import UserProfile from apps.volontulo.views import logged_as_admin @@ -51,6 +58,62 @@ def login_view(request): status=status.HTTP_400_BAD_REQUEST, ) +@api_view(['POST']) +@authentication_classes((CsrfExemptSessionAuthentication,)) +@permission_classes((AllowAny,)) +def register_view(request): + """REST API register view.""" + if not request.user.is_authenticated(): + email = request.data.get('email') + password = request.data.get('password') + + try: + user = User.objects.create_user( + username=email, + email=email, + password=password, + ) + user.is_active = False + user.save() + profile = UserProfile(user=user) + ctx = {'token': profile.uuid, 'angular_root' : settings.ANGULAR_ROOT} + profile.save() + # sending email to user: + send_mail(request, 'registration', [user.email], context=ctx) + except IntegrityError: + #return Response( + # status=status.HTTP + #) + pass + + return Response( + status=status.HTTP_201_CREATED, + ) + + return Response( + serializers.UserSerializer(request.user, context={ + 'request': request + }).data, + status=status.HTTP_400_BAD_REQUEST, + ) + +@api_view(['POST']) +@authentication_classes((CsrfExemptSessionAuthentication,)) +@permission_classes((AllowAny,)) +def activate(request): + """View responsible for activating user account.""" + try: + profile = UserProfile.objects.get(uuid=request.data.get('uuid')) + profile.user.is_active = True + profile.user.save() + return Response( + status=status.HTTP_201_CREATED, + ) + except UserProfile.DoesNotExist: + return Response( + status=status.HTTP_400_BAD_REQUEST, + ) + @api_view(['GET']) @permission_classes((AllowAny,)) diff --git a/frontend/src/app/account-activation/account-activation.component.css b/frontend/src/app/account-activation/account-activation.component.css new file mode 100644 index 00000000..e69de29b diff --git a/frontend/src/app/account-activation/account-activation.component.html b/frontend/src/app/account-activation/account-activation.component.html new file mode 100644 index 00000000..98bb1953 --- /dev/null +++ b/frontend/src/app/account-activation/account-activation.component.html @@ -0,0 +1,4 @@ + + + + \ No newline at end of file diff --git a/frontend/src/app/account-activation/account-activation.component.spec.ts b/frontend/src/app/account-activation/account-activation.component.spec.ts new file mode 100644 index 00000000..400acdc0 --- /dev/null +++ b/frontend/src/app/account-activation/account-activation.component.spec.ts @@ -0,0 +1,25 @@ +import { async, ComponentFixture, TestBed } from '@angular/core/testing'; + +import { AccountActivationComponent } from './account-activation.component'; + +describe('AccountActivationComponent', () => { + let component: AccountActivationComponent; + let fixture: ComponentFixture; + + beforeEach(async(() => { + TestBed.configureTestingModule({ + declarations: [ AccountActivationComponent ] + }) + .compileComponents(); + })); + + beforeEach(() => { + fixture = TestBed.createComponent(AccountActivationComponent); + component = fixture.componentInstance; + fixture.detectChanges(); + }); + + it('should create', () => { + expect(component).toBeTruthy(); + }); +}); diff --git a/frontend/src/app/account-activation/account-activation.component.ts b/frontend/src/app/account-activation/account-activation.component.ts new file mode 100644 index 00000000..5ccb2ec8 --- /dev/null +++ b/frontend/src/app/account-activation/account-activation.component.ts @@ -0,0 +1,39 @@ +import { Component, OnInit } from '@angular/core'; +import { AccountActivationRequestModel } from 'app/auth.models'; +import { Observable } from 'rxjs/Observable'; +import { AuthService } from '../auth.service'; +import { Router, ActivatedRoute, Params } from '@angular/router'; + +@Component({ + selector: 'volontulo-account-activation', + templateUrl: './account-activation.component.html', + styleUrls: ['./account-activation.component.css'] +}) +export class AccountActivationComponent implements OnInit { + accountActivationRequestModel: AccountActivationRequestModel = { + uuid: '', + }; + activationSuccessful: boolean = false; + + constructor(private authService: AuthService, private activatedRoute: ActivatedRoute + ) { } + + activate(): void { + this.authService.account_activation(this.accountActivationRequestModel.uuid) + .map(rsp => { + if (rsp.status === 201) + this.activationSuccessful = true; + + return Observable.of(null); + }) + .subscribe(); + } + + ngOnInit() { + this.activatedRoute.queryParams.subscribe((params: Params) => { + this.accountActivationRequestModel.uuid = params['activation_token']; + this.activate(); + }); + } + +} diff --git a/frontend/src/app/app.module.ts b/frontend/src/app/app.module.ts index bebedc23..2d736e0a 100644 --- a/frontend/src/app/app.module.ts +++ b/frontend/src/app/app.module.ts @@ -31,6 +31,7 @@ import { FaqOrganizationsComponent } from './static/faq-organizations.component' import { OfficeComponent } from './static/office/office.component'; import { FaqVolunteersComponent } from './static/faq-volunteers.component'; import { RegisterComponent } from './register/register.component'; +import { AccountActivationComponent } from './account-activation/account-activation.component'; Raven.config(environment.sentryDSN).install(); @@ -73,6 +74,10 @@ const appRoutes: Routes = [ path: 'register', component: RegisterComponent, }, + { + path: 'account-activation', + component: AccountActivationComponent + }, { path: 'regulations', component: RegulationsComponent @@ -108,6 +113,7 @@ const appRoutes: Routes = [ OfficeComponent, FaqVolunteersComponent, RegisterComponent, + AccountActivationComponent, ], imports: [ BrowserModule, diff --git a/frontend/src/app/auth.models.ts b/frontend/src/app/auth.models.ts index 900b9d54..878d70a7 100644 --- a/frontend/src/app/auth.models.ts +++ b/frontend/src/app/auth.models.ts @@ -2,3 +2,13 @@ export interface LoginRequestModel { username: string; password: string; } + +export interface RegisterRequestModel { + email: string; + password: string; + terms_acceptance: boolean; +} + +export interface AccountActivationRequestModel { + uuid: string; +} \ No newline at end of file diff --git a/frontend/src/app/auth.service.ts b/frontend/src/app/auth.service.ts index 1ba4e0a3..2dc30cdf 100644 --- a/frontend/src/app/auth.service.ts +++ b/frontend/src/app/auth.service.ts @@ -15,6 +15,8 @@ export class AuthService { private _currentUser: User; public changeUserEvent: BehaviorSubject; public resetPasswordUrl = `${environment.djangoRoot}/password-reset`; + private _registerUrl = `${environment.apiRoot}/register`; + private _accountActivationUrl = `${environment.apiRoot}/activate`; public user$: Observable; @@ -48,6 +50,28 @@ export class AuthService { }); } + register(email: string, password: string): Observable { + return this.http.post( + this._registerUrl, + { password, email }, + { withCredentials: true }) + .map(rsp => { + return rsp; + }); + } + + + account_activation(uuid: string): Observable { + return this.http.post( + this._accountActivationUrl, + { uuid }, + { withCredentials: true }) + .map(rsp => { + return rsp; + }); + } + + currentUser(): User { return this._currentUser; } diff --git a/frontend/src/app/footer/footer.component.html b/frontend/src/app/footer/footer.component.html index eba7bc06..54bae68c 100644 --- a/frontend/src/app/footer/footer.component.html +++ b/frontend/src/app/footer/footer.component.html @@ -7,7 +7,7 @@

      Informacje

      • Strona główna - Załóż konto + Załóż konto Kontakt z administratorem O nas Regulamin @@ -18,7 +18,7 @@

        Informacje

        Strefa organizacji i instytucji

        • - Załóż konto + Załóż konto Dodaj nową ofertę Lista organizacji i instytucji FAQ dla organizacji i instytucji @@ -30,7 +30,7 @@

          Strefa organizacji i instytucji

          Strefa wolontariusza

          • - Załóż konto + Załóż konto Lista aktualnych ofert Archiwum ofert Lista organizacji i instytucji diff --git a/frontend/src/app/header/header.component.html b/frontend/src/app/header/header.component.html index b31f6f68..e268cd2f 100644 --- a/frontend/src/app/header/header.component.html +++ b/frontend/src/app/header/header.component.html @@ -9,7 +9,7 @@
            • - Załóż konto + Załóż konto
            • lub diff --git a/frontend/src/app/register/register.component.html b/frontend/src/app/register/register.component.html index 68065ce1..9ffaa4e2 100644 --- a/frontend/src/app/register/register.component.html +++ b/frontend/src/app/register/register.component.html @@ -1,3 +1,49 @@ -

              - register works!ksjndk skjn -

              +

              Rejestracja w Volontulo

              + +
              +

              Email:

              + + +
              + +
              + Email is required. +
              +
              + Invalid Email. +
              + +
              +
              +
              +

              Password:

              + +
              + +
              + Password is required. +
              + +
              +
              +
              +
              + +
              +
              + +
              + Wymagana zgoda na przetwarzanie danych osobowych. +
              + +
              +
              + + +
              • \ No newline at end of file diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index 746778ce..3742f478 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -1,4 +1,8 @@ import { Component, OnInit } from '@angular/core'; +import { RegisterRequestModel } from 'app/auth.models'; +import { AuthService } from '../auth.service'; +import { Observable } from 'rxjs/Observable'; +import { RedirectComponent } from 'app/redirect.component'; @Component({ selector: 'volontulo-register', @@ -6,8 +10,28 @@ import { Component, OnInit } from '@angular/core'; styleUrls: ['./register.component.css'] }) export class RegisterComponent implements OnInit { + registerModel: RegisterRequestModel = { + email: '', + password: '', + terms_acceptance: false, + }; + ACCEPT_TERMS: string; + registrationSuccessful: boolean = false; - constructor() { } + constructor(private authService: AuthService, + ) { this.ACCEPT_TERMS = "Wyrażam zgodę na przetwarzanie moich danych osobowych" } + + register(): void { + this.authService.register(this.registerModel.email, this.registerModel.password) + .map(rsp => { + console.log(rsp.status) + if (rsp.status === 201) + this.registrationSuccessful = true; + + return Observable.of(null); + }) + .subscribe(); + } ngOnInit() { } From d95ac30f92794827c12a90be08d8a04d366a65ac Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Sun, 1 Apr 2018 17:24:32 +0200 Subject: [PATCH 05/31] MovingRegistrationToAngular --- backend/apps/volontulo/urls.py | 7 ++++++- backend/apps/volontulo/views/api.py | 15 ++++++--------- .../account-activation.component.ts | 5 ++--- frontend/src/app/auth.service.ts | 15 +++++---------- .../src/app/register/register.component.html | 1 + .../src/app/register/register.component.ts | 19 +++++++++++++------ 6 files changed, 33 insertions(+), 29 deletions(-) diff --git a/backend/apps/volontulo/urls.py b/backend/apps/volontulo/urls.py index 01b8c266..eff18b74 100644 --- a/backend/apps/volontulo/urls.py +++ b/backend/apps/volontulo/urls.py @@ -50,7 +50,12 @@ api_views.password_reset_confirm, name='password_reset_confirm' ), - url(r'^api/messages/$', api_views.messages_view, name='messages'), + url( + r'^api/messages/$', + api_views.messages_view, + name='messages' + ), + url( r'^api/register', api_views.register_view, name='api_register' diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index b3f14f88..3353d5f1 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -68,6 +68,7 @@ def login_view(request): status=status.HTTP_400_BAD_REQUEST, ) + @api_view(['POST']) @authentication_classes((CsrfExemptSessionAuthentication,)) @permission_classes((AllowAny,)) @@ -83,19 +84,15 @@ def register_view(request): email=email, password=password, ) - user.is_active = False + user.save() profile = UserProfile(user=user) - ctx = {'token': profile.uuid, 'angular_root' : settings.ANGULAR_ROOT} + ctx = {'token': profile.uuid, 'angular_root': settings.ANGULAR_ROOT} profile.save() - # sending email to user: - send_mail(request, 'registration', [user.email], context=ctx) + send_mail(request, 'registration', [user.email], context=ctx) except IntegrityError: - #return Response( - # status=status.HTTP - #) pass - + return Response( status=status.HTTP_201_CREATED, ) @@ -225,7 +222,7 @@ class OfferViewSet(viewsets.ModelViewSet): 'requirements', 'started_at', 'recruitment_end_date' - ) + ) def get_queryset(self): """Queryset depends on user role.""" diff --git a/frontend/src/app/account-activation/account-activation.component.ts b/frontend/src/app/account-activation/account-activation.component.ts index 5ccb2ec8..51956593 100644 --- a/frontend/src/app/account-activation/account-activation.component.ts +++ b/frontend/src/app/account-activation/account-activation.component.ts @@ -20,13 +20,12 @@ export class AccountActivationComponent implements OnInit { activate(): void { this.authService.account_activation(this.accountActivationRequestModel.uuid) - .map(rsp => { + .subscribe(rsp => { if (rsp.status === 201) this.activationSuccessful = true; return Observable.of(null); - }) - .subscribe(); + }); } ngOnInit() { diff --git a/frontend/src/app/auth.service.ts b/frontend/src/app/auth.service.ts index 7c19dce6..0650450d 100644 --- a/frontend/src/app/auth.service.ts +++ b/frontend/src/app/auth.service.ts @@ -9,6 +9,7 @@ import { Observable } from 'rxjs/Observable'; import { environment } from '../environments/environment'; import { User } from './user.d'; import { SuccessOrFailureAction } from './models'; +import { error } from 'selenium-webdriver'; @Injectable() @@ -101,24 +102,18 @@ export class AuthService { }); } - register(email: string, password: string): Observable { + register(email: string, password: string) { return this.http.post( this._registerUrl, { password, email }, - { withCredentials: true }) - .map(rsp => { - return rsp; - }); + { observe: 'response' }) } - account_activation(uuid: string): Observable { + account_activation(uuid: string) { return this.http.post( this._accountActivationUrl, { uuid }, - { withCredentials: true }) - .map(rsp => { - return rsp; - }); + { observe: 'response' }); } logout() { diff --git a/frontend/src/app/register/register.component.html b/frontend/src/app/register/register.component.html index 9ffaa4e2..82adef0c 100644 --- a/frontend/src/app/register/register.component.html +++ b/frontend/src/app/register/register.component.html @@ -45,5 +45,6 @@

              Rejestracja w Volontulo

      + \ No newline at end of file diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index 3742f478..080f4b93 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -3,7 +3,8 @@ import { RegisterRequestModel } from 'app/auth.models'; import { AuthService } from '../auth.service'; import { Observable } from 'rxjs/Observable'; import { RedirectComponent } from 'app/redirect.component'; - +import { Params } from '@angular/router'; +import { error } from 'selenium-webdriver'; @Component({ selector: 'volontulo-register', templateUrl: './register.component.html', @@ -17,20 +18,26 @@ export class RegisterComponent implements OnInit { }; ACCEPT_TERMS: string; registrationSuccessful: boolean = false; - + userIsAuthenticated: boolean = false; + constructor(private authService: AuthService, ) { this.ACCEPT_TERMS = "Wyrażam zgodę na przetwarzanie moich danych osobowych" } register(): void { + this.registrationSuccessful = false; + this.userIsAuthenticated = false; this.authService.register(this.registerModel.email, this.registerModel.password) - .map(rsp => { - console.log(rsp.status) + .subscribe(rsp => { if (rsp.status === 201) this.registrationSuccessful = true; return Observable.of(null); - }) - .subscribe(); + }, + error => { + if (error.status === 400) + this.userIsAuthenticated = true; + } + ); } ngOnInit() { From 2c7956f559593ad0fe5697c4efdcbd093de3ad09 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Sun, 1 Apr 2018 17:33:47 +0200 Subject: [PATCH 06/31] MovingRegistrationToAngular_2 --- backend/apps/volontulo/urls.py | 2 +- backend/apps/volontulo/views/api.py | 1 + frontend/src/app/auth.models.ts | 1 - frontend/src/app/auth.service.ts | 72 +++++++++---------- .../src/app/register/register.component.ts | 9 ++- 5 files changed, 42 insertions(+), 43 deletions(-) diff --git a/backend/apps/volontulo/urls.py b/backend/apps/volontulo/urls.py index eff18b74..28395b49 100644 --- a/backend/apps/volontulo/urls.py +++ b/backend/apps/volontulo/urls.py @@ -65,7 +65,7 @@ api_views.activate, name='api_activate' ), - + # login and loggged user space: url(r'^o/logout$', auth_views.logout, name='logout'), url(r'^o/register$', auth_views.Register.as_view(), name='register'), diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index 3353d5f1..1c7f0b0f 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -104,6 +104,7 @@ def register_view(request): status=status.HTTP_400_BAD_REQUEST, ) + @api_view(['POST']) @authentication_classes((CsrfExemptSessionAuthentication,)) @permission_classes((AllowAny,)) diff --git a/frontend/src/app/auth.models.ts b/frontend/src/app/auth.models.ts index 878d70a7..c6ce2406 100644 --- a/frontend/src/app/auth.models.ts +++ b/frontend/src/app/auth.models.ts @@ -6,7 +6,6 @@ export interface LoginRequestModel { export interface RegisterRequestModel { email: string; password: string; - terms_acceptance: boolean; } export interface AccountActivationRequestModel { diff --git a/frontend/src/app/auth.service.ts b/frontend/src/app/auth.service.ts index 0650450d..8f7081e2 100644 --- a/frontend/src/app/auth.service.ts +++ b/frontend/src/app/auth.service.ts @@ -51,14 +51,14 @@ export class AuthService { login(username: string, password: string): void { this.http.post(this.loginUrl, { username, password }) .subscribe( - user => { - this.changeUserEvent.next(user); - this.loginEvent.next({ success: true }); - this.router.navigate(['']); - }, - err => { - this.loginEvent.next({ success: false, message: err }); - } + user => { + this.changeUserEvent.next(user); + this.loginEvent.next({ success: true }); + this.router.navigate(['']); + }, + err => { + this.loginEvent.next({ success: false, message: err }); + } ); } @@ -66,40 +66,40 @@ export class AuthService { this.http.post( this.resetPasswordUrl, { username }, { observe: 'response' }) .subscribe( - response => { - if (response.status === 201) { - this.resetPasswordEvent.next({ success: true }); - } else { - this.resetPasswordEvent.next( - { - success: false, - message: `Backend return http code other than 201: ${response.status}` - }); - } - }, - err => { - this.resetPasswordEvent.next({ success: false, message: err }); - }); + response => { + if (response.status === 201) { + this.resetPasswordEvent.next({ success: true }); + } else { + this.resetPasswordEvent.next( + { + success: false, + message: `Backend return http code other than 201: ${response.status}` + }); + } + }, + err => { + this.resetPasswordEvent.next({ success: false, message: err }); + }); } confirmResetPassword(password: string, uidb64: string, token: string) { this.http.post( `${this.resetPasswordUrl}/${uidb64}/${token}`, { password }, { observe: 'response' }) .subscribe( - response => { - if (response.status === 201) { - this.confirmResetPasswordEvent.next({ success: true }); - } else { - this.confirmResetPasswordEvent.next( - { - success: false, - message: `Backend return http code other than 201: ${response.status}` - }); - } - }, - err => { - this.confirmResetPasswordEvent.next({ success: false, message: err }); - }); + response => { + if (response.status === 201) { + this.confirmResetPasswordEvent.next({ success: true }); + } else { + this.confirmResetPasswordEvent.next( + { + success: false, + message: `Backend return http code other than 201: ${response.status}` + }); + } + }, + err => { + this.confirmResetPasswordEvent.next({ success: false, message: err }); + }); } register(email: string, password: string) { diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index 080f4b93..47457cdd 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -14,12 +14,11 @@ export class RegisterComponent implements OnInit { registerModel: RegisterRequestModel = { email: '', password: '', - terms_acceptance: false, }; ACCEPT_TERMS: string; registrationSuccessful: boolean = false; userIsAuthenticated: boolean = false; - + constructor(private authService: AuthService, ) { this.ACCEPT_TERMS = "Wyrażam zgodę na przetwarzanie moich danych osobowych" } @@ -35,9 +34,9 @@ export class RegisterComponent implements OnInit { }, error => { if (error.status === 400) - this.userIsAuthenticated = true; - } - ); + this.userIsAuthenticated = true; + } + ); } ngOnInit() { From ce434e713a4bdcb60236bf9f8e5eeb3ed16a1a27 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Sun, 1 Apr 2018 17:56:28 +0200 Subject: [PATCH 07/31] MovingRegistrationToAngular_4 --- .../app/account-activation/account-activation.component.html | 2 +- frontend/src/app/auth.models.ts | 2 +- frontend/src/app/register/register.component.html | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/frontend/src/app/account-activation/account-activation.component.html b/frontend/src/app/account-activation/account-activation.component.html index 98bb1953..63fff558 100644 --- a/frontend/src/app/account-activation/account-activation.component.html +++ b/frontend/src/app/account-activation/account-activation.component.html @@ -1,4 +1,4 @@ - \ No newline at end of file + diff --git a/frontend/src/app/auth.models.ts b/frontend/src/app/auth.models.ts index c6ce2406..430e8f4c 100644 --- a/frontend/src/app/auth.models.ts +++ b/frontend/src/app/auth.models.ts @@ -10,4 +10,4 @@ export interface RegisterRequestModel { export interface AccountActivationRequestModel { uuid: string; -} \ No newline at end of file +} diff --git a/frontend/src/app/register/register.component.html b/frontend/src/app/register/register.component.html index 82adef0c..f7392f2d 100644 --- a/frontend/src/app/register/register.component.html +++ b/frontend/src/app/register/register.component.html @@ -47,4 +47,4 @@

      Rejestracja w Volontulo

      - \ No newline at end of file + From 41011c171f2bd64ba47a22aa9a3e3ed06dbf6522 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Sun, 1 Apr 2018 18:04:00 +0200 Subject: [PATCH 08/31] MovingRegistrationToAngular_5 --- backend/apps/volontulo/views/api.py | 1 + 1 file changed, 1 insertion(+) diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index 3ff3ed7b..cb4c161e 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -84,6 +84,7 @@ def register_view(request): username=email, email=email, password=password, + is_active=False ) user.save() From 662f2bd349e95bad636b0e04cf5f9b3a3b223301 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Sun, 1 Apr 2018 18:59:26 +0200 Subject: [PATCH 09/31] MovingRegistrationToAngular6 --- .../account-activation.component.ts | 10 ++++++---- .../src/app/register/register.component.ts | 19 ++++++++++++------- 2 files changed, 18 insertions(+), 11 deletions(-) diff --git a/frontend/src/app/account-activation/account-activation.component.ts b/frontend/src/app/account-activation/account-activation.component.ts index 51956593..73c6ecb9 100644 --- a/frontend/src/app/account-activation/account-activation.component.ts +++ b/frontend/src/app/account-activation/account-activation.component.ts @@ -13,17 +13,19 @@ export class AccountActivationComponent implements OnInit { accountActivationRequestModel: AccountActivationRequestModel = { uuid: '', }; - activationSuccessful: boolean = false; + activationSuccessful: boolean; constructor(private authService: AuthService, private activatedRoute: ActivatedRoute - ) { } + ) { + this.activationSuccessful = false; + } activate(): void { this.authService.account_activation(this.accountActivationRequestModel.uuid) .subscribe(rsp => { - if (rsp.status === 201) + if (rsp.status === 201) { this.activationSuccessful = true; - + } return Observable.of(null); }); } diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index 47457cdd..d097ea65 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -16,25 +16,30 @@ export class RegisterComponent implements OnInit { password: '', }; ACCEPT_TERMS: string; - registrationSuccessful: boolean = false; - userIsAuthenticated: boolean = false; + registrationSuccessful: boolean; + userIsAuthenticated: boolean; constructor(private authService: AuthService, - ) { this.ACCEPT_TERMS = "Wyrażam zgodę na przetwarzanie moich danych osobowych" } + ) { + this.ACCEPT_TERMS = 'Wyrażam zgodę na przetwarzanie moich danych osobowych'; + this.registrationSuccessful = false; + this.userIsAuthenticated = false; + } register(): void { this.registrationSuccessful = false; this.userIsAuthenticated = false; this.authService.register(this.registerModel.email, this.registerModel.password) .subscribe(rsp => { - if (rsp.status === 201) + if (rsp.status === 201) { this.registrationSuccessful = true; - + } return Observable.of(null); }, - error => { - if (error.status === 400) + err => { + if (err.status === 400) { this.userIsAuthenticated = true; + } } ); } From 6d227f41f3fdf9576e58f94971593669e04496b9 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Sun, 1 Apr 2018 19:16:22 +0200 Subject: [PATCH 10/31] MovingRegistrationToAngular7 --- backend/apps/volontulo/views/api.py | 1 - 1 file changed, 1 deletion(-) diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index cb4c161e..735b24ce 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -19,7 +19,6 @@ from django.utils.http import urlsafe_base64_encode, urlsafe_base64_decode from django_filters.rest_framework import DjangoFilterBackend from django.db.utils import IntegrityError -from django.shortcuts import redirect from rest_framework import status from rest_framework.decorators import api_view, detail_route from rest_framework.decorators import authentication_classes From cf816fc6bc06c353c50e7eab6f238ed05312bf18 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Sun, 1 Apr 2018 19:23:41 +0200 Subject: [PATCH 11/31] MovingRegistrationToAngular9 --- backend/apps/volontulo/views/api.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index 735b24ce..b79576c1 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -88,7 +88,8 @@ def register_view(request): user.save() profile = UserProfile(user=user) - ctx = {'token': profile.uuid, 'angular_root': settings.ANGULAR_ROOT} + ctx = {'token': profile.uuid, 'angular_root': + settings.ANGULAR_ROOT} profile.save() send_mail(request, 'registration', [user.email], context=ctx) except IntegrityError: From 1341cb79940fde07af421db5cab82b7f5102edd4 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 4 Apr 2018 21:55:00 +0200 Subject: [PATCH 12/31] MovingDjangoToAngular_FinalCut --- backend/apps/volontulo/forms.py | 12 -- .../apps/volontulo/templates/auth/login.html | 29 ----- .../volontulo/templates/common/footer.html | 2 +- .../templates/emails/registration.html | 2 +- .../templates/emails/registration.txt | 2 +- backend/apps/volontulo/urls.py | 16 +-- backend/apps/volontulo/views/api.py | 72 ++++++------ backend/apps/volontulo/views/auth.py | 108 ------------------ .../account-activation.component.css | 0 .../account-activation.component.html | 6 +- .../account-activation.component.ts | 40 ++++--- frontend/src/app/app.module.ts | 2 +- frontend/src/app/auth.models.ts | 3 - frontend/src/app/auth.service.ts | 12 +- frontend/src/app/login/login.component.html | 26 ++--- .../src/app/register/register.component.css | 0 .../src/app/register/register.component.html | 74 ++++++------ .../src/app/register/register.component.ts | 12 +- 18 files changed, 124 insertions(+), 294 deletions(-) delete mode 100644 backend/apps/volontulo/templates/auth/login.html delete mode 100644 frontend/src/app/account-activation/account-activation.component.css delete mode 100644 frontend/src/app/register/register.component.css diff --git a/backend/apps/volontulo/forms.py b/backend/apps/volontulo/forms.py index b7597c34..ee78b256 100644 --- a/backend/apps/volontulo/forms.py +++ b/backend/apps/volontulo/forms.py @@ -15,18 +15,6 @@ ACCEPT_TERMS = """Wyrażam zgodę na przetwarzanie moich danych osobowych""" -class UserForm(forms.ModelForm): - - """Form reposponsible for authorization.""" - email = forms.EmailField() - password = forms.CharField(widget=forms.PasswordInput()) - terms_acceptance = forms.BooleanField(label=ACCEPT_TERMS, required=True) - - class Meta(object): - model = User - fields = ['email'] - - class EditProfileForm(forms.Form): """Form reposponsible for edit user details on profile page.""" diff --git a/backend/apps/volontulo/templates/auth/login.html b/backend/apps/volontulo/templates/auth/login.html deleted file mode 100644 index 63793755..00000000 --- a/backend/apps/volontulo/templates/auth/login.html +++ /dev/null @@ -1,29 +0,0 @@ -{% extends "common/col1.html" %} - -{% block title %}Logowanie do Volontulo{% endblock %} - -{% block content %} -
      -

      Rejestracja w Volontulo

      -
      - {% csrf_token %} -
      - {{ user_form.email.errors }} - - -
      -
      - {{ user_form.password.errors }} - - -
      -
      - {{ user_form.terms_acceptance.errors }} - -
      - -
      -
      -
      -
      -{% endblock %} diff --git a/backend/apps/volontulo/templates/common/footer.html b/backend/apps/volontulo/templates/common/footer.html index 72b5a80a..64106793 100644 --- a/backend/apps/volontulo/templates/common/footer.html +++ b/backend/apps/volontulo/templates/common/footer.html @@ -6,7 +6,7 @@

      Informacje

      diff --git a/backend/apps/volontulo/templates/emails/registration.html b/backend/apps/volontulo/templates/emails/registration.html index e936f742..03ad14f4 100644 --- a/backend/apps/volontulo/templates/emails/registration.html +++ b/backend/apps/volontulo/templates/emails/registration.html @@ -11,7 +11,7 @@ Dziękujemy za rejestrację.

      Zweryfikuj swoje konto na Volontulo wchodząc w poniższy link:
      - Aktywuj + Aktywuj {% endblock %} diff --git a/backend/apps/volontulo/templates/emails/registration.txt b/backend/apps/volontulo/templates/emails/registration.txt index b66390a1..31ec1b52 100644 --- a/backend/apps/volontulo/templates/emails/registration.txt +++ b/backend/apps/volontulo/templates/emails/registration.txt @@ -2,5 +2,5 @@ {% block email_content %} Dziękujemy za rejestrację. Zweryfikuj swoje konto na Volontulo wchodząc w przeglądarce pod poniższy link: -{{ angular_root }}/account-activation?activation_token={{ token }} +{{ ANGULAR_ROOT }}/activate/{{ token }} {% endblock %} diff --git a/backend/apps/volontulo/urls.py b/backend/apps/volontulo/urls.py index 68f7e9cf..fb059c3e 100644 --- a/backend/apps/volontulo/urls.py +++ b/backend/apps/volontulo/urls.py @@ -56,24 +56,18 @@ name='messages' ), url( - r'^api/register', + r'^api/register/$', api_views.register_view, - name='api_register' + name='register' ), url( - r'^api/activate', - api_views.activate, - name='api_activate' + r'^api/activate/$', + api_views.activate_view, + name='activate' ), # login and loggged user space: url(r'^o/logout$', auth_views.logout, name='logout'), - url(r'^o/register$', auth_views.Register.as_view(), name='register'), - url( - r'^o/activate/(?P[-0-9A-Za-z]+)$', - auth_views.activate, - name='activate' - ), url(r'^o/me$', views.logged_user_profile, name='logged_user_profile'), # me/edit # me/settings diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index b79576c1..fe0e1cc0 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -4,7 +4,6 @@ .. module:: api """ -from django.conf import settings from django.contrib import messages from django.contrib.auth import authenticate from django.contrib.auth import login @@ -13,12 +12,13 @@ from django.contrib.auth.tokens import default_token_generator from django.contrib.messages import get_messages from django.contrib.sites.shortcuts import get_current_site +from django.core.exceptions import ValidationError from django.db.models import Q +from django.db.utils import IntegrityError from django.shortcuts import get_object_or_404 from django.utils.encoding import force_bytes, force_text from django.utils.http import urlsafe_base64_encode, urlsafe_base64_decode from django_filters.rest_framework import DjangoFilterBackend -from django.db.utils import IntegrityError from rest_framework import status from rest_framework.decorators import api_view, detail_route from rest_framework.decorators import authentication_classes @@ -33,9 +33,9 @@ from apps.volontulo.authentication import CsrfExemptSessionAuthentication from apps.volontulo.lib.email import send_mail from apps.volontulo.models import Organization +from apps.volontulo.models import UserProfile from apps.volontulo.serializers import \ OrganizationContactSerializer, UsernameSerializer, PasswordSerializer -from apps.volontulo.models import UserProfile from apps.volontulo.views import logged_as_admin @@ -74,55 +74,47 @@ def login_view(request): @permission_classes((AllowAny,)) def register_view(request): """REST API register view.""" - if not request.user.is_authenticated(): - email = request.data.get('email') - password = request.data.get('password') + if request.user.is_authenticated(): + return Response(status=status.HTTP_400_BAD_REQUEST) - try: - user = User.objects.create_user( - username=email, - email=email, - password=password, - is_active=False - ) + email = request.data.get('email') + password = request.data.get('password') - user.save() - profile = UserProfile(user=user) - ctx = {'token': profile.uuid, 'angular_root': - settings.ANGULAR_ROOT} - profile.save() - send_mail(request, 'registration', [user.email], context=ctx) - except IntegrityError: - pass + user = User.objects.create_user( + username=email, + email=email, + password=password, + is_active=False, + ) + try: + user.save() + except IntegrityError: + return Response(status=status.HTTP_201_CREATED) - return Response( - status=status.HTTP_201_CREATED, - ) + profile = UserProfile(user=user) + ctx = {'token': profile.uuid} + profile.save() + send_mail(request, 'registration', [user.email], context=ctx) + + return Response(status=status.HTTP_201_CREATED) - return Response( - serializers.UserSerializer(request.user, context={ - 'request': request - }).data, - status=status.HTTP_400_BAD_REQUEST, - ) @api_view(['POST']) @authentication_classes((CsrfExemptSessionAuthentication,)) @permission_classes((AllowAny,)) -def activate(request): +def activate_view(request): """View responsible for activating user account.""" try: profile = UserProfile.objects.get(uuid=request.data.get('uuid')) - profile.user.is_active = True - profile.user.save() - return Response( - status=status.HTTP_201_CREATED, - ) - except UserProfile.DoesNotExist: - return Response( - status=status.HTTP_400_BAD_REQUEST, - ) + except (UserProfile.DoesNotExist, ValidationError): + return Response(status=status.HTTP_404_NOT_FOUND) + + if profile.user.is_active: + return Response(status=status.HTTP_400_BAD_REQUEST) + profile.user.is_active = True + profile.user.save() + return Response(status=status.HTTP_201_CREATED) @api_view(['POST']) diff --git a/backend/apps/volontulo/views/auth.py b/backend/apps/volontulo/views/auth.py index ce77ed5c..aab7acd2 100644 --- a/backend/apps/volontulo/views/auth.py +++ b/backend/apps/volontulo/views/auth.py @@ -6,16 +6,7 @@ from django.conf import settings from django.contrib import auth -from django.contrib import messages -from django.contrib.auth.models import User -from django.db.utils import IntegrityError from django.shortcuts import redirect -from django.shortcuts import render -from django.views.generic import View - -from apps.volontulo.forms import UserForm -from apps.volontulo.lib.email import send_mail -from apps.volontulo.models import UserProfile def logout(request): @@ -25,102 +16,3 @@ def logout(request): """ auth.logout(request) return redirect(settings.ANGULAR_ROOT) - - -class Register(View): - """View responsible for registering new users.""" - - @staticmethod - def get(request, user_form=None): - """Simple view to render register form. - - :param request: WSGIRequest instance - :param user_form: UserForm instance - """ - if request.user.is_authenticated(): - return redirect(settings.ANGULAR_ROOT) - - return render( - request, - 'auth/login.html', - { - 'user_form': UserForm() if user_form is None else user_form, - } - ) - - @classmethod - def post(cls, request): - """Method handles creation of new user. - - :param request: WSGIRequest instance - """ - # validation of register form: - user_form = UserForm(request.POST) - if not user_form.is_valid(): - messages.error( - request, - 'Wprowadzono nieprawidłowy email, hasło lub nie wyrażono ' - 'zgody na przetwarzanie danych osobowych.' - ) - return cls.get(request, user_form) - - username = request.POST.get('email') - password = request.POST.get('password') - - ctx = {} - - # attempt of new user creation: - try: - user = User.objects.create_user( - username=username, - email=username, - password=password, - is_active=False, - ) - user.save() - profile = UserProfile(user=user) - ctx['uuid'] = profile.uuid - profile.save() - except IntegrityError: - # if attempt failed, because user already exists we need show - # error message: - messages.info( - request, - 'Użytkownik o podanym emailu już istnieje' - ) - return cls.get(request, user_form) - - # sending email to user: - send_mail(request, 'registration', [user.email], context=ctx) - - # show info about successful creation of new user and redirect to - # homepage: - messages.success( - request, - 'Rejestracja przebiegła pomyślnie' - ) - messages.info( - request, - 'Na podany przy rejestracji email został wysłany link ' - 'aktywacyjny. Aby w pełni wykorzystać konto należy je aktywować ' - 'poprzez kliknięcie linku lub wklejenie go w przeglądarce.' - ) - return redirect(settings.ANGULAR_ROOT) - - -def activate(request, uuid): - """View responsible for activating user account.""" - try: - profile = UserProfile.objects.get(uuid=uuid) - profile.user.is_active = True - profile.user.save() - messages.success( - request, - """Pomyślnie aktywowałeś użytkownika.""" - ) - except UserProfile.DoesNotExist: - messages.error( - request, - """Brak użytkownika spełniającego wymagane kryteria.""" - ) - return redirect(settings.ANGULAR_ROOT) diff --git a/frontend/src/app/account-activation/account-activation.component.css b/frontend/src/app/account-activation/account-activation.component.css deleted file mode 100644 index e69de29b..00000000 diff --git a/frontend/src/app/account-activation/account-activation.component.html b/frontend/src/app/account-activation/account-activation.component.html index 63fff558..f1917c70 100644 --- a/frontend/src/app/account-activation/account-activation.component.html +++ b/frontend/src/app/account-activation/account-activation.component.html @@ -1,4 +1,2 @@ - - - - + + diff --git a/frontend/src/app/account-activation/account-activation.component.ts b/frontend/src/app/account-activation/account-activation.component.ts index 73c6ecb9..72c3c2d8 100644 --- a/frontend/src/app/account-activation/account-activation.component.ts +++ b/frontend/src/app/account-activation/account-activation.component.ts @@ -1,5 +1,4 @@ import { Component, OnInit } from '@angular/core'; -import { AccountActivationRequestModel } from 'app/auth.models'; import { Observable } from 'rxjs/Observable'; import { AuthService } from '../auth.service'; import { Router, ActivatedRoute, Params } from '@angular/router'; @@ -7,33 +6,38 @@ import { Router, ActivatedRoute, Params } from '@angular/router'; @Component({ selector: 'volontulo-account-activation', templateUrl: './account-activation.component.html', - styleUrls: ['./account-activation.component.css'] }) export class AccountActivationComponent implements OnInit { - accountActivationRequestModel: AccountActivationRequestModel = { - uuid: '', - }; activationSuccessful: boolean; - + failureMessage: string; constructor(private authService: AuthService, private activatedRoute: ActivatedRoute ) { - this.activationSuccessful = false; } - activate(): void { - this.authService.account_activation(this.accountActivationRequestModel.uuid) - .subscribe(rsp => { - if (rsp.status === 201) { - this.activationSuccessful = true; - } - return Observable.of(null); - }); + activate(uuid: string): void { + this.authService.activateAccount(uuid) + .subscribe( + rsp => { + this.activationSuccessful = rsp.status === 201; + }, + err =>{ + this.activationSuccessful = false; + if(err.status === 400){ + this.failureMessage = 'Użytkownik został już aktywowany'; + } + else if(err.status === 404){ + this.failureMessage = 'Użytkownik nie istnieje'; + } + else{ + this.failureMessage = 'epic fail'; + } + } + ); } ngOnInit() { - this.activatedRoute.queryParams.subscribe((params: Params) => { - this.accountActivationRequestModel.uuid = params['activation_token']; - this.activate(); + this.activatedRoute.params.subscribe(params => { + this.activate(params['token']); }); } diff --git a/frontend/src/app/app.module.ts b/frontend/src/app/app.module.ts index 0012ecc7..37aa3596 100644 --- a/frontend/src/app/app.module.ts +++ b/frontend/src/app/app.module.ts @@ -85,7 +85,7 @@ const appRoutes: Routes = [ component: RegisterComponent, }, { - path: 'account-activation', + path: 'activate/:token', component: AccountActivationComponent }, { diff --git a/frontend/src/app/auth.models.ts b/frontend/src/app/auth.models.ts index 430e8f4c..af75009c 100644 --- a/frontend/src/app/auth.models.ts +++ b/frontend/src/app/auth.models.ts @@ -8,6 +8,3 @@ export interface RegisterRequestModel { password: string; } -export interface AccountActivationRequestModel { - uuid: string; -} diff --git a/frontend/src/app/auth.service.ts b/frontend/src/app/auth.service.ts index e43afc9a..6209d76f 100644 --- a/frontend/src/app/auth.service.ts +++ b/frontend/src/app/auth.service.ts @@ -9,7 +9,6 @@ import { Observable } from 'rxjs/Observable'; import { environment } from '../environments/environment'; import { User } from './user.d'; import { SuccessOrFailureAction } from './models'; -import { error } from 'selenium-webdriver'; @Injectable() @@ -18,17 +17,16 @@ export class AuthService { private loginUrl = `${environment.apiRoot}/login/`; private logoutUrl = `${environment.apiRoot}/logout/`; private resetPasswordUrl = `${environment.apiRoot}/password-reset/`; + private registerUrl = `${environment.apiRoot}/register/`; + private accountActivationUrl = `${environment.apiRoot}/activate/`; private changeUserEvent = new BehaviorSubject(null); private loginEvent = new Subject(); private resetPasswordEvent = new Subject(); private confirmResetPasswordEvent = new Subject(); - private _loginUrl = `${environment.apiRoot}/login`; private _logoutUrl = `${environment.apiRoot}/logout`; private _currentUserUrl = `${environment.apiRoot}/current-user`; private _currentUser: User; - private _registerUrl = `${environment.apiRoot}/register`; - private _accountActivationUrl = `${environment.apiRoot}/activate`; public user$: Observable = this.changeUserEvent.asObservable(); public login$: Observable = this.loginEvent.asObservable(); @@ -105,14 +103,14 @@ export class AuthService { register(email: string, password: string) { return this.http.post( - this._registerUrl, + this.registerUrl, { password, email }, { observe: 'response' }) } - account_activation(uuid: string) { + activateAccount(uuid: string) { return this.http.post( - this._accountActivationUrl, + this.accountActivationUrl, { uuid }, { observe: 'response' }); } diff --git a/frontend/src/app/login/login.component.html b/frontend/src/app/login/login.component.html index c8c6797d..ef93c6c7 100644 --- a/frontend/src/app/login/login.component.html +++ b/frontend/src/app/login/login.component.html @@ -1,16 +1,16 @@

      Logowanie

      -
      -

      Email:

      - -
      -
      -

      Password:

      - -
      -
      - Nie pamiętasz hasła? Możemy pomóc! -
      - - +
      +

      Email:

      + +
      +
      +

      Password:

      + +
      +
      + Nie pamiętasz hasła? Możemy pomóc! +
      + +
      diff --git a/frontend/src/app/register/register.component.css b/frontend/src/app/register/register.component.css deleted file mode 100644 index e69de29b..00000000 diff --git a/frontend/src/app/register/register.component.html b/frontend/src/app/register/register.component.html index f7392f2d..d28a4a05 100644 --- a/frontend/src/app/register/register.component.html +++ b/frontend/src/app/register/register.component.html @@ -1,50 +1,50 @@

      Rejestracja w Volontulo

      -
      -

      Email:

      - +
      +

      Email:

      + -
      +
      -
      - Email is required. -
      -
      - Invalid Email. -
      +
      + Email is required. +
      +
      + Invalid Email. +
      -
      -
      -

      Password:

      - -
      +
      +
      +

      Password:

      + +
      -
      - Password is required. -
      +
      + Password is required. +
      -
      -
      -
      - -
      -
      +
      +
      +
      + +
      +
      -
      - Wymagana zgoda na przetwarzanie danych osobowych. -
      +
      + Wymagana zgoda na przetwarzanie danych osobowych. +
      -
      - - - +
      + + + diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index d097ea65..d2a822d1 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -4,26 +4,22 @@ import { AuthService } from '../auth.service'; import { Observable } from 'rxjs/Observable'; import { RedirectComponent } from 'app/redirect.component'; import { Params } from '@angular/router'; -import { error } from 'selenium-webdriver'; + @Component({ selector: 'volontulo-register', templateUrl: './register.component.html', - styleUrls: ['./register.component.css'] }) export class RegisterComponent implements OnInit { registerModel: RegisterRequestModel = { email: '', password: '', }; - ACCEPT_TERMS: string; - registrationSuccessful: boolean; - userIsAuthenticated: boolean; + ACCEPT_TERMS = 'Wyrażam zgodę na przetwarzanie moich danych osobowych'; + registrationSuccessful = false; + userIsAuthenticated = false; constructor(private authService: AuthService, ) { - this.ACCEPT_TERMS = 'Wyrażam zgodę na przetwarzanie moich danych osobowych'; - this.registrationSuccessful = false; - this.userIsAuthenticated = false; } register(): void { From 90b69de855e5fdae23df0352867d7dfa81b50d6d Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 18 Apr 2018 20:48:52 +0200 Subject: [PATCH 13/31] MovingDjangoToAngular_Corrections --- backend/apps/volontulo/forms.py | 2 - backend/apps/volontulo/urls.py | 9 ++- backend/apps/volontulo/views/api.py | 4 +- .../account-activation.component.html | 2 - .../app/activation/activation.component.html | 2 + .../activation.component.spec.ts} | 12 +-- .../activation.component.ts} | 11 +-- frontend/src/app/app.module.ts | 6 +- frontend/src/app/auth.service.ts | 79 +++++++++---------- .../src/app/register/register.component.html | 36 ++++----- .../src/app/register/register.component.ts | 8 +- frontend/src/sass/styles.scss | 6 ++ 12 files changed, 88 insertions(+), 89 deletions(-) delete mode 100644 frontend/src/app/account-activation/account-activation.component.html create mode 100644 frontend/src/app/activation/activation.component.html rename frontend/src/app/{account-activation/account-activation.component.spec.ts => activation/activation.component.spec.ts} (50%) rename frontend/src/app/{account-activation/account-activation.component.ts => activation/activation.component.ts} (80%) diff --git a/backend/apps/volontulo/forms.py b/backend/apps/volontulo/forms.py index ee78b256..82522d3d 100644 --- a/backend/apps/volontulo/forms.py +++ b/backend/apps/volontulo/forms.py @@ -12,8 +12,6 @@ from apps.volontulo.models import UserGallery from apps.volontulo.utils import get_administrators_emails -ACCEPT_TERMS = """Wyrażam zgodę na przetwarzanie moich danych osobowych""" - class EditProfileForm(forms.Form): diff --git a/backend/apps/volontulo/urls.py b/backend/apps/volontulo/urls.py index fb059c3e..bd932d6a 100644 --- a/backend/apps/volontulo/urls.py +++ b/backend/apps/volontulo/urls.py @@ -60,12 +60,19 @@ api_views.register_view, name='register' ), + # url( + # r'^api/activate/$', + # api_views.activate_view, + # name='activate' + # ), + url( - r'^api/activate/$', + r'^api/activate/(?P[-0-9A-Za-z]+)/$', api_views.activate_view, name='activate' ), + # login and loggged user space: url(r'^o/logout$', auth_views.logout, name='logout'), url(r'^o/me$', views.logged_user_profile, name='logged_user_profile'), diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index fe0e1cc0..2e90d94e 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -103,10 +103,10 @@ def register_view(request): @api_view(['POST']) @authentication_classes((CsrfExemptSessionAuthentication,)) @permission_classes((AllowAny,)) -def activate_view(request): +def activate_view(request, uuid): """View responsible for activating user account.""" try: - profile = UserProfile.objects.get(uuid=request.data.get('uuid')) + profile = UserProfile.objects.get(uuid=uuid) except (UserProfile.DoesNotExist, ValidationError): return Response(status=status.HTTP_404_NOT_FOUND) diff --git a/frontend/src/app/account-activation/account-activation.component.html b/frontend/src/app/account-activation/account-activation.component.html deleted file mode 100644 index f1917c70..00000000 --- a/frontend/src/app/account-activation/account-activation.component.html +++ /dev/null @@ -1,2 +0,0 @@ - - diff --git a/frontend/src/app/activation/activation.component.html b/frontend/src/app/activation/activation.component.html new file mode 100644 index 00000000..26f000ed --- /dev/null +++ b/frontend/src/app/activation/activation.component.html @@ -0,0 +1,2 @@ + + diff --git a/frontend/src/app/account-activation/account-activation.component.spec.ts b/frontend/src/app/activation/activation.component.spec.ts similarity index 50% rename from frontend/src/app/account-activation/account-activation.component.spec.ts rename to frontend/src/app/activation/activation.component.spec.ts index 400acdc0..5d082e9c 100644 --- a/frontend/src/app/account-activation/account-activation.component.spec.ts +++ b/frontend/src/app/activation/activation.component.spec.ts @@ -1,20 +1,20 @@ import { async, ComponentFixture, TestBed } from '@angular/core/testing'; -import { AccountActivationComponent } from './account-activation.component'; +import { ActivationComponent } from './activation.component'; -describe('AccountActivationComponent', () => { - let component: AccountActivationComponent; - let fixture: ComponentFixture; +describe('ActivationComponent', () => { + let component: ActivationComponent; + let fixture: ComponentFixture; beforeEach(async(() => { TestBed.configureTestingModule({ - declarations: [ AccountActivationComponent ] + declarations: [ ActivationComponent ] }) .compileComponents(); })); beforeEach(() => { - fixture = TestBed.createComponent(AccountActivationComponent); + fixture = TestBed.createComponent(ActivationComponent); component = fixture.componentInstance; fixture.detectChanges(); }); diff --git a/frontend/src/app/account-activation/account-activation.component.ts b/frontend/src/app/activation/activation.component.ts similarity index 80% rename from frontend/src/app/account-activation/account-activation.component.ts rename to frontend/src/app/activation/activation.component.ts index 72c3c2d8..fc9006f5 100644 --- a/frontend/src/app/account-activation/account-activation.component.ts +++ b/frontend/src/app/activation/activation.component.ts @@ -4,10 +4,10 @@ import { AuthService } from '../auth.service'; import { Router, ActivatedRoute, Params } from '@angular/router'; @Component({ - selector: 'volontulo-account-activation', - templateUrl: './account-activation.component.html', + selector: 'volontulo-activation', + templateUrl: './activation.component.html', }) -export class AccountActivationComponent implements OnInit { +export class ActivationComponent implements OnInit { activationSuccessful: boolean; failureMessage: string; constructor(private authService: AuthService, private activatedRoute: ActivatedRoute @@ -20,7 +20,8 @@ export class AccountActivationComponent implements OnInit { rsp => { this.activationSuccessful = rsp.status === 201; }, - err =>{ + err => { + this.activationSuccessful = false; if(err.status === 400){ this.failureMessage = 'Użytkownik został już aktywowany'; @@ -29,7 +30,7 @@ export class AccountActivationComponent implements OnInit { this.failureMessage = 'Użytkownik nie istnieje'; } else{ - this.failureMessage = 'epic fail'; + this.failureMessage = 'Coś poszło nie tak, skontaktuj się z administratorem'; } } ); diff --git a/frontend/src/app/app.module.ts b/frontend/src/app/app.module.ts index 37aa3596..7e15dbc6 100644 --- a/frontend/src/app/app.module.ts +++ b/frontend/src/app/app.module.ts @@ -41,7 +41,7 @@ import { PasswordResetComponent } from './password-reset/password-reset.componen import { PasswordResetConfirmComponent } from './password-reset/password-reset-confirm.component'; import { OrganizationOffersListComponent } from './organization/organization-offers-list/organization-offers-list.component'; import { RegisterComponent } from './register/register.component'; -import { AccountActivationComponent } from './account-activation/account-activation.component'; +import { ActivationComponent } from './activation/activation.component'; Raven.config(environment.sentryDSN).install(); @@ -86,7 +86,7 @@ const appRoutes: Routes = [ }, { path: 'activate/:token', - component: AccountActivationComponent + component: ActivationComponent }, { path: 'regulations', @@ -144,7 +144,7 @@ registerLocaleData(localePl); MessagesComponent, OrganizationOffersListComponent, RegisterComponent, - AccountActivationComponent, + ActivationComponent, ], imports: [ BrowserModule.withServerTransition({ appId: 'volontulo' }), diff --git a/frontend/src/app/auth.service.ts b/frontend/src/app/auth.service.ts index 6209d76f..0d1439c7 100644 --- a/frontend/src/app/auth.service.ts +++ b/frontend/src/app/auth.service.ts @@ -18,13 +18,12 @@ export class AuthService { private logoutUrl = `${environment.apiRoot}/logout/`; private resetPasswordUrl = `${environment.apiRoot}/password-reset/`; private registerUrl = `${environment.apiRoot}/register/`; - private accountActivationUrl = `${environment.apiRoot}/activate/`; + private activationUrl = `${environment.apiRoot}/activate/`; private changeUserEvent = new BehaviorSubject(null); private loginEvent = new Subject(); private resetPasswordEvent = new Subject(); private confirmResetPasswordEvent = new Subject(); - private _logoutUrl = `${environment.apiRoot}/logout`; private _currentUserUrl = `${environment.apiRoot}/current-user`; private _currentUser: User; @@ -50,14 +49,14 @@ export class AuthService { login(username: string, password: string): void { this.http.post(this.loginUrl, { username, password }) .subscribe( - user => { - this.changeUserEvent.next(user); - this.loginEvent.next({ success: true }); - this.router.navigate(['']); - }, - err => { - this.loginEvent.next({ success: false, message: err }); - } + user => { + this.changeUserEvent.next(user); + this.loginEvent.next({ success: true }); + this.router.navigate(['']); + }, + err => { + this.loginEvent.next({ success: false, message: err }); + } ); } @@ -65,40 +64,40 @@ export class AuthService { this.http.post( this.resetPasswordUrl, { username }, { observe: 'response' }) .subscribe( - response => { - if (response.status === 201) { - this.resetPasswordEvent.next({ success: true }); - } else { - this.resetPasswordEvent.next( - { - success: false, - message: `Backend return http code other than 201: ${response.status}` - }); - } - }, - err => { - this.resetPasswordEvent.next({ success: false, message: err }); - }); + response => { + if (response.status === 201) { + this.resetPasswordEvent.next({ success: true }); + } else { + this.resetPasswordEvent.next( + { + success: false, + message: `Backend return http code other than 201: ${response.status}` + }); + } + }, + err => { + this.resetPasswordEvent.next({ success: false, message: err }); + }); } confirmResetPassword(password: string, uidb64: string, token: string) { this.http.post( `${this.resetPasswordUrl}/${uidb64}/${token}`, { password }, { observe: 'response' }) .subscribe( - response => { - if (response.status === 201) { - this.confirmResetPasswordEvent.next({ success: true }); - } else { - this.confirmResetPasswordEvent.next( - { - success: false, - message: `Backend return http code other than 201: ${response.status}` - }); - } - }, - err => { - this.confirmResetPasswordEvent.next({ success: false, message: err }); - }); + response => { + if (response.status === 201) { + this.confirmResetPasswordEvent.next({ success: true }); + } else { + this.confirmResetPasswordEvent.next( + { + success: false, + message: `Backend return http code other than 201: ${response.status}` + }); + } + }, + err => { + this.confirmResetPasswordEvent.next({ success: false, message: err }); + }); } register(email: string, password: string) { @@ -110,8 +109,8 @@ export class AuthService { activateAccount(uuid: string) { return this.http.post( - this.accountActivationUrl, - { uuid }, + `${this.activationUrl}${uuid}/`, + null, { observe: 'response' }); } diff --git a/frontend/src/app/register/register.component.html b/frontend/src/app/register/register.component.html index d28a4a05..076e0d0b 100644 --- a/frontend/src/app/register/register.component.html +++ b/frontend/src/app/register/register.component.html @@ -2,40 +2,32 @@

      Rejestracja w Volontulo

      Email:

      - - -
      +
      - Email is required. + Pole 'Email' jest wymagane.
      -
      - Invalid Email. +
      + Nieprawidłowy Email.
      -
      -

      Password:

      - -
      +

      Hasło:

      + +
      -
      - Password is required. -
      + Pole 'Hasło' jest wymagane.
      -
      -
      - -
      +
      + +
      diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index d2a822d1..0d20ff80 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -1,4 +1,4 @@ -import { Component, OnInit } from '@angular/core'; +import { Component } from '@angular/core'; import { RegisterRequestModel } from 'app/auth.models'; import { AuthService } from '../auth.service'; import { Observable } from 'rxjs/Observable'; @@ -9,7 +9,7 @@ import { Params } from '@angular/router'; selector: 'volontulo-register', templateUrl: './register.component.html', }) -export class RegisterComponent implements OnInit { +export class RegisterComponent { registerModel: RegisterRequestModel = { email: '', password: '', @@ -39,8 +39,4 @@ export class RegisterComponent implements OnInit { } ); } - - ngOnInit() { - } - } diff --git a/frontend/src/sass/styles.scss b/frontend/src/sass/styles.scss index 6d76f8e5..4fc6fed7 100644 --- a/frontend/src/sass/styles.scss +++ b/frontend/src/sass/styles.scss @@ -17,3 +17,9 @@ html { font-size: $base-font-size; } + +/* Temporary hack. We need to raise bootstrap version.*/ + +.form-check { + padding-left: $form-check-input-gutter; +} From a463dcb40e5155075c27fecb33eeadd8880c143f Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 18 Apr 2018 21:13:29 +0200 Subject: [PATCH 14/31] YetAnotherUpdate --- frontend/src/app/activation/activation.component.ts | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/frontend/src/app/activation/activation.component.ts b/frontend/src/app/activation/activation.component.ts index fc9006f5..8263b8d7 100644 --- a/frontend/src/app/activation/activation.component.ts +++ b/frontend/src/app/activation/activation.component.ts @@ -23,13 +23,11 @@ export class ActivationComponent implements OnInit { err => { this.activationSuccessful = false; - if(err.status === 400){ + if (err.status === 400) { this.failureMessage = 'Użytkownik został już aktywowany'; - } - else if(err.status === 404){ + } else if (err.status === 404) { this.failureMessage = 'Użytkownik nie istnieje'; - } - else{ + } else { this.failureMessage = 'Coś poszło nie tak, skontaktuj się z administratorem'; } } From 6d1e9a749c472d8d7d396a8adfd5d90ac89c9e67 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 18 Apr 2018 21:33:25 +0200 Subject: [PATCH 15/31] ANdAnotherUpdate --- backend/apps/volontulo/urls.py | 6 ------ 1 file changed, 6 deletions(-) diff --git a/backend/apps/volontulo/urls.py b/backend/apps/volontulo/urls.py index bd932d6a..10866b5a 100644 --- a/backend/apps/volontulo/urls.py +++ b/backend/apps/volontulo/urls.py @@ -60,12 +60,6 @@ api_views.register_view, name='register' ), - # url( - # r'^api/activate/$', - # api_views.activate_view, - # name='activate' - # ), - url( r'^api/activate/(?P[-0-9A-Za-z]+)/$', api_views.activate_view, From e195cba36ae3d2eae37cd4e8f587bf6506dfa470 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 9 May 2018 17:53:40 +0200 Subject: [PATCH 16/31] CommitBeforeMergingUpstream20180509 --- .gitignore | 4 - Makefile | 41 ++- backend/.dockerignore | 1 - backend/Dockerfile | 33 +- backend/apps/volontulo/factories.py | 2 +- backend/apps/volontulo/permissions.py | 17 +- backend/apps/volontulo/serializers.py | 16 + .../templates/offers/offers_list.html | 2 +- .../templates/users/user_profile.html | 2 +- .../views/api/organizations/test_create.py | 16 +- .../views/api/organizations/test_update.py | 24 +- backend/apps/volontulo/views/offers.py | 4 +- backend/docker-entrypoint.sh | 8 - backend/requirements/base.txt | 1 - backend/volontulo_org/settings/dev.py | 7 - fabfile.py | 9 - frontend/src/app/banner/banner.component.html | 2 +- frontend/src/app/banner/banner.component.ts | 3 +- .../cookie-law-banner.component.html | 22 +- .../cookie-law-banner.component.scss | 11 +- .../organization-details.component.html | 8 +- .../organization/organization.component.ts | 2 +- .../organization/organization.interfaces.ts | 5 + .../app/organization/organization.service.ts | 29 +- .../organizations.component.html | 9 +- .../faq-organizations.component.spec.ts | 4 +- .../app/static/faq-organizations.component.ts | 75 ++++- .../static/faq-volunteers.component.spec.ts | 2 + .../app/static/faq-volunteers.component.ts | 281 +++++++++++++++++- frontend/src/sass/_variables.scss | 4 +- 30 files changed, 509 insertions(+), 135 deletions(-) diff --git a/.gitignore b/.gitignore index 5854e642..e1ac754c 100644 --- a/.gitignore +++ b/.gitignore @@ -53,9 +53,6 @@ coverage.xml # Django stuff: *.log -# Sphinx documentation -docs/_build/ - # PyBuilder target/ @@ -67,7 +64,6 @@ target/ backend/apps/volontulo/node_modules/ backend/apps/volontulo/static/ backend/media -backend/static frontend/node_modules/ frontend/tmp/ diff --git a/Makefile b/Makefile index e7f3fb50..8695120a 100644 --- a/Makefile +++ b/Makefile @@ -1,9 +1,38 @@ -make test_backend: - docker-compose run --rm backend python3 manage.py test && pylint apps +# Test commands +test_backend: + docker-compose run --rm backend python3 manage.py test + docker-compose run --rm backend pylint apps -make test_frontend: +test_frontend: docker-compose run --rm frontend node_modules/.bin/ng test --single-run -make test_all: - make test_backend - make test_frontend +test_all: test_backend test_frontend + + +# Lint commands +lint_backend: + docker-compose run --rm backend /bin/bash -c \ + "echo -e '\n\nRunning pycodestyle...';\ + pycodestyle --exclude='apps/volontulo/migrations/*,node_modules,.ropeproject' .;\ + echo -e '\nRunning pylint...';\ + pylint apps;" + +lint_frontend: + docker-compose run --rm frontend node_modules/.bin/ng lint + +lint_all: lint_backend lint_frontend + + +# Docker commands +docker_remove_containers: + docker rm $$(docker ps -qa --filter name='volontulo_*') + +docker_remove_images: + docker rmi $$(docker images -qa --filter reference='*/volontulo*') + +docker_remove_all: docker_remove_containers docker_remove_images + + +# Other commands +populate_database: + docker-compose run --rm backend python3 manage.py populate_database diff --git a/backend/.dockerignore b/backend/.dockerignore index 31f44370..8f8a13ad 100644 --- a/backend/.dockerignore +++ b/backend/.dockerignore @@ -7,4 +7,3 @@ coverage.xml apps/volontulo/node_modules/ media/ fake_emails/ -static/ diff --git a/backend/Dockerfile b/backend/Dockerfile index 09fbf438..e56896a9 100644 --- a/backend/Dockerfile +++ b/backend/Dockerfile @@ -6,43 +6,24 @@ EXPOSE 8000 # Docker-based quirks ENV LANG C.UTF-8 RUN rm /bin/sh && ln -s /bin/bash /bin/sh -RUN echo "deb-src http://archive.ubuntu.com/ubuntu/ xenial main restricted" >> /etc/apt/sources.list # Docker env variables -ENV NODE_VERSION 9.3.0 -ENV NVM_DIR /usr/local/nvm -ENV NODE_PATH $NVM_DIR/versions/node/v$NODE_VERSION/lib/node_modules -ENV PATH $NVM_DIR/versions/node/v$NODE_VERSION/bin:$PATH ENV USER_DIR="/root" # System-based package isntallation -RUN apt-get update -y -RUN apt-get install -y python3-pip \ - python-virtualenv \ - libpq-dev \ - wget +RUN apt-get update -y && apt-get install -y python3-pip python-virtualenv libpq-dev && apt-get autoremove + +# Setup wait-for-it for docker-compose +ADD ./wait-for-it.sh /usr/bin/wait-for-it +RUN chmod a+x /usr/bin/wait-for-it # Backend setup +RUN pip3 install --upgrade pip + ADD requirements /volontulo/backend/requirements WORKDIR /volontulo/backend -RUN pip3 install --upgrade pip RUN pip3 install -r ./requirements/dev.txt -# Old fronted setup -RUN wget -qO- https://raw.githubusercontent.com/creationix/nvm/v0.33.1/install.sh | bash -RUN . $NVM_DIR/nvm.sh && \ - nvm install $NODE_VERSION && \ - nvm alias default $NODE_VERSION && \ - nvm use default - -# Setup wait-for-it for docker-compose -RUN wget -qO /usr/bin/wait-for-it https://raw.githubusercontent.com/vishnubob/wait-for-it/master/wait-for-it.sh -RUN chmod a+x /usr/bin/wait-for-it - # Add build-time directories ADD . /volontulo/backend - -WORKDIR /volontulo/backend/apps/volontulo -RUN npm install -RUN node ./node_modules/.bin/gulp build WORKDIR /volontulo/backend diff --git a/backend/apps/volontulo/factories.py b/backend/apps/volontulo/factories.py index ccef853d..373bcd5d 100644 --- a/backend/apps/volontulo/factories.py +++ b/backend/apps/volontulo/factories.py @@ -164,7 +164,7 @@ class Meta: # pylint: disable=C0111 is_main = True path = ImageField(from_path=os.path.join( os.path.dirname(__file__), - 'frontend/img/volontulo_baner.png' + 'static/volontulo/img/volontulo_baner.png' )) diff --git a/backend/apps/volontulo/permissions.py b/backend/apps/volontulo/permissions.py index 3432251d..d4399fab 100644 --- a/backend/apps/volontulo/permissions.py +++ b/backend/apps/volontulo/permissions.py @@ -33,5 +33,18 @@ class OrganizationPermission(permissions.BasePermission): """REST API organizations permissions.""" def has_permission(self, request, view): - """We are accepting only safe methods for now.""" - return request.method in permissions.SAFE_METHODS + """We are accepting safe methods, post an put methods only for + authenticated users """ + return request.method in permissions.SAFE_METHODS or ( + request.method in ('POST', 'PUT') and + request.user.is_authenticated() + ) + + def has_object_permission(self, request, view, obj): + user = request.user + return request.method in permissions.SAFE_METHODS or ( + user.is_authenticated() and ( + user.userprofile.is_administrator or + obj in user.userprofile.organizations.all() + ) + ) diff --git a/backend/apps/volontulo/serializers.py b/backend/apps/volontulo/serializers.py index 3de29002..a9617d67 100644 --- a/backend/apps/volontulo/serializers.py +++ b/backend/apps/volontulo/serializers.py @@ -38,6 +38,12 @@ def get_slug(obj): """Returns slugified name.""" return slugify(obj.name) + def create(self, validated_data): + """Adds created organization to user's organizations list""" + org = super(OrganizationSerializer, self).create(validated_data) + org.userprofiles.add(self.context['request'].user.userprofile) + return org + class OrganizationField(serializers.Field): @@ -187,10 +193,15 @@ class UserSerializer(serializers.ModelSerializer): is_administrator = serializers.SerializerMethodField() organizations = serializers.SerializerMethodField() + phone_no = serializers.SerializerMethodField() class Meta: model = User fields = ( + 'first_name', + 'last_name', + 'phone_no', + 'email', 'is_administrator', 'organizations', 'username', @@ -206,6 +217,11 @@ def get_organizations(self, obj): qs = obj.userprofile.organizations.all() return OrganizationSerializer(qs, many=True, context=self.context).data + @staticmethod + def get_phone_no(obj): + """Returns user's phone number.""" + return obj.userprofile.phone_no + # pylint: disable=abstract-method class OrganizationContactSerializer(serializers.Serializer): diff --git a/backend/apps/volontulo/templates/offers/offers_list.html b/backend/apps/volontulo/templates/offers/offers_list.html index 5ee733bc..1ef1efd2 100644 --- a/backend/apps/volontulo/templates/offers/offers_list.html +++ b/backend/apps/volontulo/templates/offers/offers_list.html @@ -36,7 +36,7 @@

      Lista ofert

      -
      {{ offer.organization.name }}
      +
      {{ offer.organization.name }}
      {% if user.userprofile.is_administrator %} diff --git a/backend/apps/volontulo/templates/users/user_profile.html b/backend/apps/volontulo/templates/users/user_profile.html index f874fa34..2b96acd1 100644 --- a/backend/apps/volontulo/templates/users/user_profile.html +++ b/backend/apps/volontulo/templates/users/user_profile.html @@ -71,7 +71,7 @@

      Organizowane oferty

      {% else %}

      Pracujesz w organizacji zajmującej się wolontariatem?

      - Dodaj organizację + Dodaj organizację
      {% endif %}
      diff --git a/backend/apps/volontulo/tests/views/api/organizations/test_create.py b/backend/apps/volontulo/tests/views/api/organizations/test_create.py index 4713b5ef..99fd0c7e 100644 --- a/backend/apps/volontulo/tests/views/api/organizations/test_create.py +++ b/backend/apps/volontulo/tests/views/api/organizations/test_create.py @@ -17,11 +17,11 @@ class _TestOrganizationsCreateAPIView(TestOffersCommons, APITestCase): def setUp(self): """Set up each test.""" super(_TestOrganizationsCreateAPIView, self).setUp() - self.organization_payload = ( - b'{"name": "TM",' - b'"slug": "tm",' - b'"url": "http://localhost:8000/api/organizations/1/"}' - ) + self.organization_payload = b"""{ + "name": "TM", + "description": "Opis", + "address": "ul. Koperkowa 7" + }""" class TestAdminUserOrganizationsCreateAPIView(_TestOrganizationsCreateAPIView): @@ -44,7 +44,7 @@ def test_organization_create_status(self): content_type='application/json', ) - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + self.assertEqual(response.status_code, status.HTTP_201_CREATED) class TestOrganizationUserOrganizationsCreateAPIView( @@ -71,7 +71,7 @@ def test_organization_create_status(self): content_type='application/json', ) - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + self.assertEqual(response.status_code, status.HTTP_201_CREATED) class TestRegularUserOrganizationsCreateAPIView( @@ -98,7 +98,7 @@ def test_organization_create_status(self): content_type='application/json', ) - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + self.assertEqual(response.status_code, status.HTTP_201_CREATED) class TestAnonymousUserOrganizationsCreateAPIView( diff --git a/backend/apps/volontulo/tests/views/api/organizations/test_update.py b/backend/apps/volontulo/tests/views/api/organizations/test_update.py index f5f64f8d..6e181285 100644 --- a/backend/apps/volontulo/tests/views/api/organizations/test_update.py +++ b/backend/apps/volontulo/tests/views/api/organizations/test_update.py @@ -17,11 +17,11 @@ class _TestOrganizationsUpdateAPIView(TestOffersCommons, APITestCase): def setUp(self): """Set up each test.""" super(_TestOrganizationsUpdateAPIView, self).setUp() - self.organization_payload = ( - b'{"name": "TM",' - b'"slug": "tm",' - b'"url": "http://localhost:8000/api/organizations/1/"}' - ) + self.organization_payload = b"""{ + "name": "TM", + "description": "Opis", + "address": "ul. Koperkowa 7" + }""" class TestAdminUserOrganizationsUpdateAPIView(_TestOrganizationsUpdateAPIView): @@ -39,12 +39,12 @@ def test_organization_update_status(self): API for now is read-only. """ response = self.client.put( - '/api/organizations/1/', + '/api/organizations/{}/'.format(self.organization.id), self.organization_payload, content_type='application/json', ) - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + self.assertEqual(response.status_code, status.HTTP_200_OK) class TestOrganizationUserOrganizationsUpdateAPIView( @@ -66,12 +66,11 @@ def test_organization_update_status(self): API for now is read-only. """ response = self.client.put( - '/api/organizations/1/', + '/api/organizations/{}/'.format(self.organization.id), self.organization_payload, content_type='application/json', ) - - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + self.assertEqual(response.status_code, status.HTTP_200_OK) class TestRegularUserOrganizationsUpdateAPIView( @@ -93,11 +92,10 @@ def test_organization_update_status(self): API for now is read-only. """ response = self.client.put( - '/api/organizations/1/', + '/api/organizations/{}/'.format(self.organization.id), self.organization_payload, content_type='application/json', ) - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) @@ -112,7 +110,7 @@ def test_organization_update_status(self): API for now is read-only. """ response = self.client.put( - '/api/organizations/1/', + '/api/organizations/{}/'.format(self.organization.id), self.organization_payload, content_type='application/json', ) diff --git a/backend/apps/volontulo/views/offers.py b/backend/apps/volontulo/views/offers.py index a8ff3747..d5fedf4e 100644 --- a/backend/apps/volontulo/views/offers.py +++ b/backend/apps/volontulo/views/offers.py @@ -94,9 +94,9 @@ def get(request): messages.info(request, mark_safe( "Nie masz jeszcze żadnej założonej organizacji" " na {0}. Aby założyć organizację," - " kliknij tu.".format( + " kliknij tu.".format( settings.SYSTEM_DOMAIN, - reverse('organizations_create') + settings.ANGULAR_ROOT, ) )) return redirect('offers_list') diff --git a/backend/docker-entrypoint.sh b/backend/docker-entrypoint.sh index e726abf5..1c6209eb 100644 --- a/backend/docker-entrypoint.sh +++ b/backend/docker-entrypoint.sh @@ -1,14 +1,6 @@ #!/bin/bash set -e -# we need to re-create static directory during creation of container because -# mounted volume overwrites all files available during build phase: -if [ ! -f apps/volontulo/static ]; then - cd apps/volontulo; - node node_modules/.bin/gulp build; - cd -; -fi - wait-for-it db:5432 python3 manage.py migrate diff --git a/backend/requirements/base.txt b/backend/requirements/base.txt index bdfd47fb..00701043 100644 --- a/backend/requirements/base.txt +++ b/backend/requirements/base.txt @@ -1,7 +1,6 @@ Django==1.11.5 django-bootstrap3==9.0.0 django-cookie-law==1.0.13 -logilab-common==1.4.1 Pillow==4.2.1 six==1.11.0 unipath==1.1 diff --git a/backend/volontulo_org/settings/dev.py b/backend/volontulo_org/settings/dev.py index 1e26ee9b..8012112a 100644 --- a/backend/volontulo_org/settings/dev.py +++ b/backend/volontulo_org/settings/dev.py @@ -11,19 +11,12 @@ 'backend', ] -DEBUG_TOOLBAR_PATCH_SETTINGS = False - INSTALLED_APPS += ( - 'debug_toolbar', 'django_coverage', 'django_extensions', 'django_nose', ) -MIDDLEWARE_CLASSES += ( - 'debug_toolbar.middleware.DebugToolbarMiddleware', -) - TEST_RUNNER = 'django_nose.NoseTestSuiteRunner' EMAIL_BACKEND = 'django.core.mail.backends.filebased.EmailBackend' diff --git a/fabfile.py b/fabfile.py index 2114f0bd..b1e66968 100644 --- a/fabfile.py +++ b/fabfile.py @@ -56,21 +56,12 @@ def update(): ): run('pip install --upgrade -r requirements/base.txt') - # Gulp frontend refresh: - with contextlib.nested( - prefix('nvm use {}'.format(NODE_VERSION)), - cd('/var/www/volontulo/backend/apps/volontulo'), - ): - run('npm install .') - run('node node_modules/.bin/gulp build') - # Django site refresh: with contextlib.nested( cd('/var/www/volontulo/backend'), prefix('workon volontulo') ): run('python manage.py migrate --traceback') - run('python manage.py collectstatic --traceback --noinput') # Angular assets refresh: with contextlib.nested( diff --git a/frontend/src/app/banner/banner.component.html b/frontend/src/app/banner/banner.component.html index 622e6c79..116d3603 100644 --- a/frontend/src/app/banner/banner.component.html +++ b/frontend/src/app/banner/banner.component.html @@ -1 +1 @@ -Volontulo Baner +Volontulo Baner diff --git a/frontend/src/app/banner/banner.component.ts b/frontend/src/app/banner/banner.component.ts index 765fe3cc..23d75807 100644 --- a/frontend/src/app/banner/banner.component.ts +++ b/frontend/src/app/banner/banner.component.ts @@ -2,7 +2,6 @@ import { Component } from '@angular/core'; @Component({ selector: 'volontulo-banner', - templateUrl: './banner.component.html', - styleUrls: ['./banner.component.scss'] + templateUrl: './banner.component.html' }) export class BannerComponent { } diff --git a/frontend/src/app/cookie-law-banner/cookie-law-banner.component.html b/frontend/src/app/cookie-law-banner/cookie-law-banner.component.html index 389e8cb1..b6dbdbb6 100644 --- a/frontend/src/app/cookie-law-banner/cookie-law-banner.component.html +++ b/frontend/src/app/cookie-law-banner/cookie-law-banner.component.html @@ -1,14 +1,18 @@
      Informacja o cookies
      -

      - Nasza strona zapisuje niewielkie pliki tekstowe, nazywane ciasteczkami (ang. cookies) na Twoim urządzeniu w celu - lepszego dostosowania treści oraz dla celów statystycznych. Możesz wyłączyć możliwość ich zapisu, zmieniając - ustawienia Twojej przeglądarki. Korzystanie z naszej strony bez zmiany ustawień oznacza zgodę na przechowywanie - cookies w Twoim urządzeniu. - -

      +
      +

      + Nasza strona zapisuje niewielkie pliki tekstowe, nazywane ciasteczkami (ang. cookies) na Twoim urządzeniu w celu + lepszego dostosowania treści oraz dla celów statystycznych. Możesz wyłączyć możliwość ich zapisu, zmieniając + ustawienia Twojej przeglądarki. Korzystanie z naszej strony bez zmiany ustawień oznacza zgodę na przechowywanie + cookies w Twoim urządzeniu. +

      +
      + +
      +
      diff --git a/frontend/src/app/cookie-law-banner/cookie-law-banner.component.scss b/frontend/src/app/cookie-law-banner/cookie-law-banner.component.scss index 582f492e..946958ea 100644 --- a/frontend/src/app/cookie-law-banner/cookie-law-banner.component.scss +++ b/frontend/src/app/cookie-law-banner/cookie-law-banner.component.scss @@ -1,10 +1,7 @@ +@import "~sass/variables"; + .cookie-law-banner { - background: #fff; - color: #656D78; - font-size: 0.8em; + color: gray('600'); + font-size: $font-size-sm; box-shadow: inset 0 -20px 20px -20px rgba(67, 74, 84, 0.1); } - -.cookie-law-button { - margin-left: 10px; -} diff --git a/frontend/src/app/organization/organization-details/organization-details.component.html b/frontend/src/app/organization/organization-details/organization-details.component.html index 9386ec5c..b546d30f 100644 --- a/frontend/src/app/organization/organization-details/organization-details.component.html +++ b/frontend/src/app/organization/organization-details/organization-details.component.html @@ -15,9 +15,9 @@

      Organizacja

      -
      - Edytuj organizację - Dodaj ofertę -
      +
      + Edytuj organizację + Dodaj ofertę
      +
      diff --git a/frontend/src/app/organization/organization.component.ts b/frontend/src/app/organization/organization.component.ts index 87a2ee41..67e14ec1 100644 --- a/frontend/src/app/organization/organization.component.ts +++ b/frontend/src/app/organization/organization.component.ts @@ -35,7 +35,7 @@ export class OrganizationComponent implements OnInit { this.activatedRoute.params.subscribe( params => { - this.organizationService.getOrganization(params.organizationId); + this.organizationService.getOrganization(params.organizationId).subscribe(); this.organizationService.getOffersForOrganization(params.organizationId); } ); diff --git a/frontend/src/app/organization/organization.interfaces.ts b/frontend/src/app/organization/organization.interfaces.ts index d56c6a39..05e7dc12 100644 --- a/frontend/src/app/organization/organization.interfaces.ts +++ b/frontend/src/app/organization/organization.interfaces.ts @@ -2,3 +2,8 @@ export interface ContactStatus { data: any; status: 'success' | 'error'; } + +export interface CreateOrEditOrganization { + data: any; + type: 'create' | 'edit' | 'error'; +} diff --git a/frontend/src/app/organization/organization.service.ts b/frontend/src/app/organization/organization.service.ts index 3c3e6e48..97ec1d56 100644 --- a/frontend/src/app/organization/organization.service.ts +++ b/frontend/src/app/organization/organization.service.ts @@ -6,7 +6,7 @@ import { Subject } from 'rxjs/Subject'; import { Observable } from 'rxjs/Observable'; import { Organization, OrganizationContactPayload } from './organization.model'; -import { ContactStatus } from './organization.interfaces'; +import { ContactStatus, CreateOrEditOrganization } from './organization.interfaces'; import { Offer } from '../homepage-offer/offers.model'; import { loadDefaultImage } from '../homepage-offer/offer.utils'; import { environment } from '../../environments/environment'; @@ -19,17 +19,22 @@ export class OrganizationService { private organizationEvent = new BehaviorSubject(null); private organizationsEvent = new Subject(); private offersEvent = new Subject(); + private createOrganizationEvent = new Subject(); public contactStatus$: Observable = this.contactStatusEvent.asObservable(); public organization$: Observable = this.organizationEvent.asObservable(); public organizations$: Observable = this.organizationsEvent.asObservable(); public offers$: Observable = this.offersEvent.asObservable(); + public createOrEditOrganization$: Observable = this.createOrganizationEvent.asObservable(); constructor(private http: HttpClient) { } - getOrganization(id: number) { + getOrganization(id: number): Observable { return this.http.get(`${this.url}${id}/`) - .subscribe(organization => this.organizationEvent.next(organization)); + .map(organization => { + this.organizationEvent.next(organization); + return organization; + }); } sendContactForm(organization: Organization, contactData: OrganizationContactPayload) { @@ -49,6 +54,24 @@ export class OrganizationService { ); } + createOrganization(newOrganization: Organization) { + this.http.post(this.url, newOrganization) + .subscribe( + (data: HttpResponse) => { + this.createOrganizationEvent.next({data, type: 'create'}); + }, + error => {this.createOrganizationEvent.next({ data: error.error, type: 'error'}) + }); + } + editOrganization(id: number, updatedOrganization: Organization) { + this.http.put(`${this.url}${id}/`, updatedOrganization) + .subscribe( (data: HttpResponse) => { + this.createOrganizationEvent.next({data, type: 'edit'}); + }, + error => { + this.createOrganizationEvent.next({ data: error.error, type: 'error'}); + }); + } getOrganizationViewUrl(organization: Organization): string { return `${environment.djangoRoot}/organizations/${organization.slug}/${organization.id}`; } diff --git a/frontend/src/app/organizations/organizations.component.html b/frontend/src/app/organizations/organizations.component.html index 8c1e5937..bef52857 100644 --- a/frontend/src/app/organizations/organizations.component.html +++ b/frontend/src/app/organizations/organizations.component.html @@ -1,13 +1,12 @@
      -
      - Dodaj organizację +
      + Dodaj organizację

      Lista organizacji

      -
      @@ -30,7 +29,7 @@

      Lista organizacji

      Brak zdefiniowanych organizacji.

       - +
      -
      +
      diff --git a/frontend/src/app/static/faq-organizations.component.spec.ts b/frontend/src/app/static/faq-organizations.component.spec.ts index e59d05ca..2379aa03 100644 --- a/frontend/src/app/static/faq-organizations.component.spec.ts +++ b/frontend/src/app/static/faq-organizations.component.spec.ts @@ -1,4 +1,5 @@ import { async, ComponentFixture, TestBed } from '@angular/core/testing'; +import { NgbAccordionModule } from '@ng-bootstrap/ng-bootstrap'; import { FaqOrganizationsComponent } from './faq-organizations.component'; @@ -8,7 +9,8 @@ describe('FaqOrganizationsComponent', () => { beforeEach(async(() => { TestBed.configureTestingModule({ - declarations: [ FaqOrganizationsComponent ] + imports: [ NgbAccordionModule.forRoot() ], + declarations: [ FaqOrganizationsComponent ], }) .compileComponents(); })); diff --git a/frontend/src/app/static/faq-organizations.component.ts b/frontend/src/app/static/faq-organizations.component.ts index 08020587..70da31b9 100644 --- a/frontend/src/app/static/faq-organizations.component.ts +++ b/frontend/src/app/static/faq-organizations.component.ts @@ -2,6 +2,77 @@ import { Component } from '@angular/core'; @Component({ selector: 'volontulo-faq-organizations', - templateUrl: './faq-organizations.component.html', + templateUrl: './faq.component.html', }) -export class FaqOrganizationsComponent {} + +export class FaqOrganizationsComponent { + title = 'organizatorów'; + faqs: { + question: string, + answer: string, + }[] = [ + { + question: `Czy wolontariuszem może być osoba niepełnoletnia?`, + answer: `W przypadku osób, które nie są pełnoletnie na rozpoczęcie działalności woluntarystycznej wymagana jest zgoda przedstawiciela + ustawowego (zazwyczaj – rodziców). Organizacje chcące korzystać z pracy niepełnoletnich wolontariuszy, aby uniknąć ewentualnych + nieprzyjemności powinny prosić kandydatów o przedstawienie pisemnej zgody rodziców. W placówkach opiekuńczo – wychowawczych + (zaliczamy do nich domy dziecka i świetlice środowiskowe), wolontariuszami mogą być tylko osoby pełnoletnie.`, + }, + { + question: `Na rzecz jakich podmiotów wolontariusze mogą wykonywać świadczenia?`, + answer: `Ustawa o działalności pożytku publicznego i o wolontariacie w art. 42 ust. 1 określa krąg podmiotów uprawnionych do +zawierania porozumień z wolontariuszami, zwanych w ustawie korzystającymi. Są nimi: +
        +
      • organizacje pozarządowe oraz inne podmioty prowadzące działalność pożytku publicznego – w zakresie ich działalności + statutowej (a nie gospodarczej), w szczególności w zakresie działalności pożytku publicznego,
        • +
      • jednostki organizacyjne podległe organom administracji publicznej lub nadzorowane przez te organy – z wyłączeniem + prowadzonej przez te jednostki działalności gospodarczej,
        • +
      • organy administracji publicznej – z wyłączeniem prowadzadzonej przez nie działalności gospodarczej.
        • +
      `, + }, + { + question: `Czy wolontariusz może działać na rzecz przedsiębiorców?`, + answer: `Podmioty, na rzecz których wolontariusz może wykonywać świadczenia zostały określone powyżej. Ponadto, ze względu na cel + i charakter świadczeń wolontariuszy nie jest dopuszczalne, aby wykonywali oni świadczenia na rzecz przedsiębiorców i innych + podmiotów prowadzących działalność gospodarczą. Wolontariusze nie powinni również zastępować pracowników a jedynie uzupełniać + ich pracę.`, + }, + { + question: `Czy wolontariusz jest pracownikiem?`, + answer: `Wolontariusz nie jest pracownikiem, ponieważ, za wykonanie świadczenia na rzecz korzystającego, wolontariusz nie pobiera + wynagrodzenia i wykonuje je na podstawie porozumienia, a nie umowy o pracę (art.3 w zw. z art.42 ustawy). Ponadto wykonywane przez + wolontariusza świadczenie nie jest świadczeniem pracy, lecz świadczeniem odpowiadającym świadczeniu pracy.` + }, + { + question: `Czy wolontariusz musi mieć podpisane porozumienie?`, + answer: `Wolontariusz pracuje w organizacji na podstawie i w oparciu o porozumienie zawarte z organizacją. W przypadku wolontariatu + przekraczającego 30 dni porozumienie musi mieć formę pisemną. Dla porozumień obejmujących wolontariat krótkoterminowy (poniżej 30 + dni) wystarczy forma ustna. Jednakże zawsze na żądania wolontariusza porozumienie musi zostać zawarte na piśmie. Porozumienie + powinno zawierać zakres, sposób i czas wykonywania świadczenia, a także postanowienie o możliwości jego rozwiązania.`, + }, + { + question: `Jakie obowiązki będą spoczywały na podmiotach uprawnionych do zawierania z wolontariuszami porozumień?`, + answer: `Z art. 45 ustawy wynika, że korzystający jest zobowiązany do zapewnienia odpowiednich środków ochrony indywidualnej + uzależnionych od rodzaju świadczeń i zagrożeń związanych z ich wykonywaniem. Korzystający ma również obowiązek informowania + wolontariusza o ryzyku dla zdrowia i bezpieczeństwa związanym z wykonywanymi świadczeniami oraz o zasadach ochrony przed + zagrożeniami, jak również do zapewnienia wolontariuszom bezpiecznych i higienicznych warunków wykonywania świadczeń.`, + }, + { + question: `Czy członek/członkini stowarzyszenia może być wolontariuszem w swojej organizacji?`, + answer: `Tak. 12 marca 2010 roku weszły w życie zmiany w dotychczasowej ustawie o działalności pożytku publicznego i wolontariacie. + Jedna z nich mówi, że członek lub członkini swojej organizacji może być w niej jednocześnie wolontariuszem.`, + }, + { + question: `Czy Szkolny Klub Wolontariusza musi być gdzieś zarejestrowany?`, + answer: `Nie`, + }, + { + question: `Jak można nagradzać swoich wolontariuszy?`, + answer: `Jest wiele form i możliwości nagradzania swoich wolontariuszy. W zależności od zasobów i środków, którymi dysponujemy, + mogą być to np.: listy gratulacyjne, dyplomy, zaproszenia na dodatkowe warsztaty, zaproszenia na spotkania zespołu pracowników, + informacja na stronie internetowej, pochwała przed innymi wolontariuszami, powierzanie coraz bardziej odpowiedzialnych zadań, + wdrażanie pomysłów wolontariuszy, wpis do księgi pamiątkowej, impreza z okazji Dnia Wolontariusza, wysyłanie kartek + okolicznościowych, drobne gadżety, własne stanowisko pracy itd.`, + }, + ]; +} diff --git a/frontend/src/app/static/faq-volunteers.component.spec.ts b/frontend/src/app/static/faq-volunteers.component.spec.ts index bc995f37..90a0b37f 100644 --- a/frontend/src/app/static/faq-volunteers.component.spec.ts +++ b/frontend/src/app/static/faq-volunteers.component.spec.ts @@ -1,4 +1,5 @@ import { async, ComponentFixture, TestBed } from '@angular/core/testing'; +import { NgbAccordionModule } from '@ng-bootstrap/ng-bootstrap'; import { FaqVolunteersComponent } from './faq-volunteers.component'; @@ -8,6 +9,7 @@ describe('FaqVolunteersComponent', () => { beforeEach(async(() => { TestBed.configureTestingModule({ + imports: [ NgbAccordionModule.forRoot() ], declarations: [ FaqVolunteersComponent ] }) .compileComponents(); diff --git a/frontend/src/app/static/faq-volunteers.component.ts b/frontend/src/app/static/faq-volunteers.component.ts index d0fcbee3..be6804f5 100644 --- a/frontend/src/app/static/faq-volunteers.component.ts +++ b/frontend/src/app/static/faq-volunteers.component.ts @@ -1,14 +1,277 @@ -import { Component, OnInit } from '@angular/core'; +import { Component } from '@angular/core'; @Component({ selector: 'volontulo-faq-volunteers', - templateUrl: './faq-volunteers.component.html', + templateUrl: './faq.component.html', }) -export class FaqVolunteersComponent implements OnInit { - - constructor() { } - - ngOnInit() { - } - +export class FaqVolunteersComponent { + title = 'wolontariuszy'; + faqs: { + question: string, + answer: string, + }[] = [ + { + question: `Czym jest wolontariat?`, + answer: `Nazwa pochodzi od łacińskiego słowa volontarius, które można przetłumaczyć jako “dobrowolny” lub + “chętny”. Centrum Wolontariatu definiuje wolontariat jako świadomą, dobrowolną działalność podejmowaną na rzecz + innych, wykraczającą poza więzi rodzinno – przyjacielsko – koleżeńskie. Wolontariuszem jest każda osoba fizyczna, + która dobrowolnie, ochotniczo i bez wynagrodzenia wykonuje świadczenia na rzecz organizacji, instytucji lub + osób indywidualnych wykraczając poza więzi koleżeńsko-rodzinne. Zasady wolontariatu zostały opisane w ustawie + o działalności pożytku publicznego i o wolontariacie uchwalonej w 2003 roku.` + }, + { + question: `Kim jest wolontariusz?`, + answer: `To ludzie, którzy dobrowolnie i bez wynagrodzenia niosą pomoc, angażują się w pracę na rzecz osób i + instytucji działających w różnych obszarach życia społecznego. Można ich spotkać między innymi w domach dziecka, + hospicjach, domach pomocy społecznej, muzeach i schroniskach dla zwierząt. Pracują w instytucjach publicznych, + organizacjach pozarządowych, placówkach kultury, sportu i wielu innych. Wolontariuszem może być każdy, w każdej + dziedzinie życia społecznego, wszędzie tam, gdzie taka pomoc jest potrzebna. Kandydat na wolontariusza nie musi + posiadać ściśle kreślonych kompetencji, kwalifikacji (chyba, że organizacja lub instytucja wskazuje na nie + w ogłoszeniu) - wystarczające są dobre chęci i wolny czas.`, + }, + { + question: `Jakie są motywacje, aby stać się wolontariuszem?`, + answer: `Motywacje, które decydują o zostaniu wolontariuszem to m.in.: +
        +
      • chęć zrobienia czegoś dobrego, pożytecznego,
        • +
      • potrzeba kontaktu z ludźmi lub nawiązania nowych znajomości
        • +
      • chęć bycia potrzebnym
        • +
      • chęć spłacenia dobra, które kiedyś od kogoś się otrzymało
        • +
      • chęć zdobycia nowych umiejętności oraz doświadczeń zawodowych i życiowych
        • +
      +

      Wolontariat jest także ważny ponieważ:

      +
        +
      • uczy wrażliwości na drugiego człowieka,
        • +
      • pomaga przetrwać okres bezrobocia,
        • +
      • jest dobrym przygotowaniem do przyszłej pracy
        • +
      • daje możliwość oderwania się od monotonii codziennego życia.
        • +
      `, + }, + { + question: `Czy wolontariuszem może być osoba niepełnoletnia? Czy mój wiek jako wolontariusza jest istotny?`, + answer: `Wolontariuszem może zostać każdy bez względu na swój wiek. Jednak niektóre formy wolontariatu wymagają +od wolontariusza pełnoletności. Wolontariat jest świadczony na podstawieumowy, którą wolontariusze zawierają z osobą +reprezentującą podmiot, na rzecz którego będą działali. Zasady zawierania umów reguluje Kodeks Cywilny, który bardzo +szczegółowo opisuje, w jaki sposób umowy mogą być zawierane. Osoby niepełnoletnie nie mogą samodzielnie zawrzeć takiej +umowy. W przypadku osoby, która nie ukończyła 13 roku życia, umowę taka może w jej imieniu zawrzeć rodzic lub prawny +opiekun. Jeśli osoba ukończyła 13 rok życia, to umowę taką może zawrzeć samodzielnie, ale tylko za zgodą rodzica lub +prawnego opiekuna – wtedy osoba chcąca zostać wolontariuszem musi przedstawić taką zgodę, najczęściej w formie pisemnej. + Organizacje chcące korzystać z pracy niepełnoletnich wolontariuszy, aby uniknąć ewentualnych nieprzyjemności, powinny + prosić kandydatów o przedstawienie pisemnej zgody rodziców.
      W placówkach opiekuńczo – wychowawczych (zaliczamy + do nich domy dziecka i świetlice środowiskowe) wolontariuszami mogą być tylko osoby pełnoletnie. W niektórych przypadkach + wolontariat może być zadaniem bardzo wymagającym, dlatego zgodnie z prawem w niektórych miejscach wolontariusze mogą + być tylko osoby pełnoletnie. Obecnie takie placówki to: Publiczne poradnie psychologiczno-pedagogiczne w tym publiczne poradnie + specjalistyczne (podstawa prawna: Rozporządzenie Ministra Edukacji Narodowej z dnia 1 lutego 2013 r. w sprawie szczegółowych zasad + działania publicznych poradni psychologiczno-pedagogicznych, w tym publicznych poradni specjalistycznych, Dz. U. nr. 0 poz. 199).`, + }, + { + question: `Czy wolontariusz musi posiadać szczególne uprawnienia?`, + answer: `W zasadzie wolontariuszem może zostać każda osoba bez szczególnych uprawnień. Jednakże, jeżeli wolontariusz + ma wykonywać świadczenie, dla wykonywania którego przepisy wymagają np. określonych badań, uprawnień etc., to wolontariusz powinien + takie badania przejść lub też posiadać stosowne uprawnienia. Wymóg ten wynika ze szczególnego charakteru niektórych + zadań, które mogą być powierzane przez organizację wolontariuszowi, m.in.: w zakresie pomocy społecznej, ochrony zdrowia, + działania na rzecz osób niepełnosprawnych. Wolontariusz musi posiadać kwalifikacje tylko wtedy, gdy obowiązek ten wynika + z przepisów prawa.`, + }, + { + question: `Ile czasu muszę poświęcić jako wolontariusz?`, + answer: `Wolontariat może być zajęciem wykonywanym codziennie, jak i raz w tygodniu. Warto się zastanowić, czy ze + względu na inne obowiązki bardziej odpowiedni dla nas jest wolontariat akcyjny, krótkoterminowy czy długoterminowy. + To wszystko jest bardzo ważne, ponieważ pozwoli jasno doprecyzować zakres obowiązków danego wolontariusza, a także + będzie istotną informacją dla organizacji lub instytucji - kiedy można liczyć na pomoc danej osoby. Precyzyjne + określenie dyspozycyjności przez wolontariusza pozwoli lepiej dopasować jego oczekiwania do potrzeb organizacji + lub instytucji.`, + }, + { + question: `Jakie wolontariusz ma prawa?`, + answer: `Ustawa o działalności pożytku publicznego i o wolontariacie daje wolontariuszom szereg uprawnień. +Poniżej najważniejsze z nich. Wolontariusz ma prawo do: +
        +
      • okresu próbnego pozwalającego poznać specyfikę nowej pracy,
        • +
      • otrzymywania wymaganych środków ochrony osobistej,
        • +
      • bycia poinformowanym o ryzyku dla zdrowia i bezpieczeństwa związanym z wykonywaniem świadczeń, a + także o zasadach ochrony przed zagrożeniem,
        • +
      • zapewnienia, na zasadach takich jak pracownicy, bezpiecznych i higienicznych warunków pracy,
        • +
      • zwrotu poniesionych kosztów podróży służbowych i diet związanych ze świadczeniem,
        • +
      • uzyskania pisemnego potwierdzenia treści porozumienia,
        • +
      • otrzymania zaświadczenia o wykonaniu świadczeń,
        • +
      • zaopatrzenia z tytułu wykonywania świadczeń – innymi słowy prawo do odszkodowania, a nawet renty + inwalidzkiej, jeżeli podczas wykonywania świadczeń ulegnie wypadkowi,
        • +
      • zwrotu pokrycia kosztów ogólnie przyjętych wynikających z umów międzynarodowych, gdy w porozumieniu + jest mowa o oddelegowaniu wolontariusza na terytorium innego państwa
        • +
      • bycia poinformowanym o swoich prawach i obowiązkach.
        • +
      `, + }, + { + question: `Jakie wolontariusz ma obowiązki?`, + answer: `Zgodnie z Ustawą o działalności pożytku publicznego i o wolontariacie, wolontariusz powinien posiadać + kwalifikacje i spełniać wymagania odpowiednie do rodzaju i zakresu świadczonej pomocy, jeżeli taki obowiązek wynika + z przepisów prawa (np. w przypadku pomocy medycznej osobom bezdomnym lub w przypadku wolontariuszy na stanowisku + nauczyciela). Jest on również zobligowany do wywiązania się z obowiązków określonych w porozumieniu, nawet jeśli nie + przybrało ono formy pisemnej. Jest odpowiedzialny również za dbanie o powierzony majątek, mienie rzeczowe, a także + przestrzegać np. tajemnicy służbowej.`, + }, + { + question: `Co to jest etyka pracy wolontariusza?`, + answer: `Decydując się na działalność w charakterze wolontariusza, należy przestrzegać pewnych zasad. Nie są +one zapisane w żadnych ustawach, kodeksach czy innych przepisach. To nieformalny zbiór norm postępowania, którymi +każdy wolontariusz powinien się kierować. Wśród nich znajdują się takie normy jak m.in.: +
        +
      • będę wypełniać wszystkie zadania związane z przyjętą rolą;
        • +
      • nie będę składać obietnic, których nie jestem w stanie spełnić;
        • +
      • w przypadku niemożności wywiązania się ze zobowiązań, poinformuję o tym koordynatora pracy + wolontariuszy;
        • +
      • zachowam dyskrecję w sprawach prywatnych, będę unikać zachowań, które mogą być niewłaściwie + rozumiane;
        • +
      • będę otwarty na nowe pomysły i sposoby działania;
        • +
      • wykorzystam szansę poznania i nauczenia się nowych rzeczy od innych osób;
        • +
      • nie będę krytykować rzeczy, których nie rozumiem;
        • +
      • będę pytać o rzeczy, których nie rozumiem;
        • +
      • będę działać w zespole;
        • +
      • będę osobą, na której można polegać;
        • +
      • będę pracować lepiej i z większą satysfakcją wykonując to, czego się ode mnie oczekuje;
        • +
      • będę chętnie się uczyć; wiem, że nauka jest nieodłączną częścią każdej dobrze wykonanej pracy;
        • +
      • będę uczestniczyć w obowiązkowych spotkaniach;
        • +
      • postaram się być bardzo dobrym wolontariuszem.
        • +
      `, + }, + { + question: `Gdzie wolontariusz może pomagać?`, + answer: `Wolontariuszem można być wszędzie z wyłączeniem sektora biznesu. Miejsca, w których można świadczyć pracę + wolontarystyczną to m.in.: stowarzyszenia, fundacje, instytucje, które są organami administracji publicznej + (ministerstwa, urzędy miejskie), wszystkie jednostki organizacyjne, które podlegają organom administracji publicznej + (m.in. ośrodki pomocy społecznej, biblioteki, muzea, szkoły, uczelnie, przedszkola, szpitale publiczne. Inne miejsca, + które mogą angażować wolontariuszy to choćby organizacje kościelne, stowarzyszenia jednostek samorządu terytorialnego, + organizacje pracodawców, kluby sportowe i policja. Jeżeli natomiast kogoś interesuje pomoc osobom fizycznym – + indywidualnym (np. wielodzietnej rodzinie z bloku obok czy dziecku ze szkoły osiedlowej, które ma kłopoty w nauce), + tu sprawa wygląda nieco inaczej. Można to oczywiście zrobić, ale musimy pamiętać o jednym: osoba indywidualna nie ma + takich uprawnień jak organizacje bądź instytucje, nie może podpisać z nami porozumienia o współpracy, a więc nie może + samodzielnie skorzystać z pomocy wolontariusza. Może to zrobić tylko za pośrednictwem wyżej wymienionych placówek.`, + }, + { + question: `Czy wolontariusz dostaje wynagrodzenie?`, + answer: `Wolontariusz wykonuje świadczenia na rzecz korzystającego ochotniczo i bez wynagrodzenia. Z powyższego wynika, że + wolontariusz za wykonywane przez siebie świadczenia nie może otrzymywać wynagrodzenia.`, + }, + { + question: `Czym się różni wolontariat zagraniczny od krajowego?`, + answer: `Staże i wolontariaty odbywane za granicą różnią się od tych realizowanych w miejscu zamieszkania przede + wszystkim kosztami. Na całym świecie organizacje przyjmujące wolontariuszy nie płacą im za ich pracę i zaangażowanie, + nie opłacają też ich przyjazdu, zakwaterowania, wyżywienia i ubezpieczenia. Wszystkie koszty ponosi sam wolontariusz.`, + }, + { + question: `Czy można być wolontariuszem za granicą?`, + answer: `Tak. Osoba, która chce zostać wolontariuszem za granicą musi być pełnoletnia i znać język angielski lub język kraju, + do którego zamierza się wybrać. Najczęściej poszukiwane są osoby mające już jakiekolwiek doświadczenie wolontarystyczne, ale nie + jest to obowiązek. Jeśli wolontariusz udaje się do egotycznych krajów (np. w Afryce, Azji, czy Ameryce Południowej), powinien + zainteresować się, czy nie musi poddać się odpowiednim szczepieniom. Polecamy zasięgnąć opinii i wiedzy u organizacji zajmujących + się wolontariatem za granicą.`, + }, + { + question: `Czy cudzoziemcy mogą być wolontariuszami?`, + answer: `Wolontariuszami mogą być nie tylko obywatele polscy, ale także cudzoziemcy legalnie przebywający w Polsce. + Zgodnie z art. 42 ust.1 świadczenie wolontariuszy jest świadczeniem odpowiadającym świadczeniu pracy, nie jest natomiast + świadczeniem pracy lub usług. W związku z tym wolontariusz – cudzoziemiec nie musi uzyskiwać zezwolenia na pracę aby + zostać wolontariuszem.`, + }, + { + question: `Czy mogę zamienić praktyki studenckie czy staż na wolontariat?`, + answer: `Wolontariatem nie są praktyki studenckie ani staż. Praktyki studenckie są formą zdobywania praktycznej wiedzy + związanej z kierunkiem studiów. Ich odbycie jest obowiązkiem studenta wynikającym np. z regulaminu uczelni, a więc + praktyka studencka, choć bezpłatna - nie jest dobrowolna i nie jest wolontariatem. A wolontariusz to osoba, która + ochotniczo i bez wynagrodzenia wykonuje świadczenia odpowiadające świadczeniom pracy, wykraczając poza obszar swoich + codziennych obowiązków.`, + }, + { + question: `Czy okres pracy wolontarystycznej liczy się do stażu pracy?`, + answer: `Nie. Tak samo wykonywanie świadczeń przez wolontariusza nie wpływa na uprawnienia pracownicze. Nie można więc + okresu pracy wolontariusza wliczyć do okresów składkowych lub nieskładkowych, co w efekcie nie ma wpływu na uzyskanie + prawa do renty, czy emerytury ani tym bardziej na wysokość obu tych świadczeń.`, + }, + { + question: `Czy wolontariusz jest pracownikiem?`, + answer: `Nie. Świadczenie wykonywane przez wolontariusza jest „świadczeniem odpowiadającym świadczeniu pracy”, +nie jest to praca w rozumieniu Kodeksu Pracy. Są jeszcze dwie istotne różnice: +
        +
      • wolontariusz za swoją pracę nie pobiera wynagrodzenia;
        • +
      • wolontariusz podpisuje porozumienie o współpracy, a nie umowę o pracę.
        • +
      `, + }, + { + question: `Czy osoba bezrobotna może być wolontariuszem bez utraty prawa do zasiłku?`, + answer: ` Tak. Bycie wolontariuszem nie wpływa ani na utratę, ani na wysokość świadczenia dla bezrobotnych. + Wolontariusz nie jest pracownikiem w rozumieniu Kodeksu Pracy, a więc nie straci prawa do zasiłku.`, + }, + { + question: `Czy podmioty gospodarcze mogą korzystać z pomocy wolontariuszy?`, + answer: `Ustawa nie przewiduje możliwości angażowania wolontariuszy przez podmioty gospodarcze. Ze względu na cel + i charakter świadczeń wolontariuszy nie jest dopuszczalne, aby wykonywali oni świadczenia na rzecz przedsiębiorców i + innych podmiotów prowadzących działalność gospodarczą. Wolontariusze nie powinni również zastępować pracowników a jedynie + uzupełniać ich pracę. Oznacza to, że w żadnej sytuacji firmy, sklepy, kancelarie prawne czy biura rachunkowe nie mogą + korzystać z pomocy wolontariuszy. Nawet, gdy podejmują działania prospołeczne.`, + }, + { + question: `Czy wolontariusz musi mieć podpisane porozumienie?`, + answer: `Wolontariusz pracuje w organizacji na podstawie i w oparciu o porozumienie zawarte z organizacją. W przypadku + wolontariatu przekraczającego 30 dni porozumienie musi mieć formę pisemną. Dla porozumień obejmujących wolontariat krótkoterminowy + (poniżej 30 dni) wystarczy forma ustna. Jednakże zawsze na żądania wolontariusza porozumienie musi zostać zawarte na piśmie.`, + }, + { + question: `Co powinno zawierać porozumienie?`, + answer: `Porozumienie powinno zawierać zakres, sposób i czas wykonywania świadczenia, a także postanowienie o możliwości jego + rozwiązania.`, + }, + { + question: `Czy wolontariusz w trakcie lub po zakończeniu świadczenia może zażądać wydania pisemnego zaświadczenia?`, + answer: `Tak. Takie zaświadczenie powinno zawierać informację o tym, co wolontariusz robił w ramach swoich obowiązków oraz + jakie zdobył doświadczenie. Organizacja ma obowiązek wystawić zaświadczenie na każdą prośbę wolontariusza.`, + }, + { + question: `Czy organizacja ma obowiązek pokrywania kosztów podróży służbowych i diet wolontariusza?`, + answer: `Tak, organizacja ma obowiązek pokrywania kosztów podróży służbowych i diet wolontariusza a zasadach dotyczących + pracowników. Jednakże wolontariusz może się zrzec tego uprawnienia podpisując odpowiednie oświadczenie.`, + }, + { + question: `Kiedy wolontariuszowi przysługuje ubezpieczenie od Następstw Nieszczęśliwych Wypadków?`, + answer: `Ubezpieczenie NNW organizacja zobowiązana jest pokrywać w razie zawarcia umowy na czas krótszy niż 30 dni. W tym +wypadku organizacja musi wykupić dla swoich wolontariuszy ubezpieczenie NW. Jeżeli natomiast organizacja zawiera porozumienie na +czas dłuższy niż 30 dni i na czas określony to wolontariusz zostaje objęty ubezpieczeniem na mocy tzw. małej ustawy wypadkowej. +
      W wypadku zawarcia umowy z wolontariuszem na czas nieokreślony organizacja musi wykupić mu ubezpieczenie NW przez pierwszych +30 dni. Od początku drugiego miesiąca wykonywania świadczeń przez wolontariusza zostaje on objęty ubezpieczeniem na mocy małej +ustawy wypadkowej.`, + }, + { + question: `Czy świadczenia wolontarystyczne mogą być traktowane jako darowizna?`, + answer: `Nie. Świadczenia wolontarystyczne nie mogą być traktowane jako darowizna.`, + }, + { + question: `Czy członek/członkini stowarzyszenia może być wolontariuszem w swojej organizacji?`, + answer: `Tak. 12 marca 2010 roku weszły w życie zmiany w dotychczasowej ustawie o działalności pożytku publicznego i wolontariacie. + Jedna z nich mówi, że członek lub członkini swojej organizacji może być w niej jednocześnie wolontariuszem.`, + }, + { + question: `Czy organizacja ma obowiązek wysyłania wolontariusza na dodatkowe szkolenia z zakresu działalności, jakiej się podejmuje + jako wolontariusz?`, + answer: `Nie. Jednakże jeżeli organizacja uzna, że ma na takie szkolenie środki i uważa udział w szkoleniu za zasadny, to może + zapewnić wolontariuszowi udział w interesujących szkoleniach.`, + }, + { + question: `Czy Szkolny Klub Wolontariusza musi być gdzieś zarejestrowany?`, + answer: `Nie.`, + }, + { + question: `Czy w organizacji przyjmującej wolontariuszy musi być koordynator pracy wolontariuszy?`, + answer: `Nie jest to obowiązek prawny. Natomiast doświadczenie podpowiada, że w organizacji powinna być wyznaczona osoba, + która będzie koordynowała pracę wolontariuszy, przeprowadzała wstępną rozmowę, wprowadzi wolontariuszy w życie organizacji, + wskaże zakres zadań, dba o relacje z wolontariuszami oraz o jakość świadczeń.`, + }, + { + question: `Jak można nagradzać swoich wolontariuszy?`, + answer: `Wiele form i możliwości nagradzania swoich wolontariuszy. W zależności od zasobów i środków, którymi dysponujemy, + mogą być to np.: listy gratulacyjne, dyplomy, zaproszenia na dodatkowe warsztaty, zaproszenia na spotkania zespołu pracowników, + informacja na stronie internetowej, pochwała przed innymi wolontariuszami, powierzanie coraz bardziej odpowiedzialnych zadań, + wdrażanie pomysłów wolontariuszy, wpis do księgi pamiątkowej, impreza z okazji Dnia Wolontariusza, wysyłanie kartek + okolicznościowych, drobne gadżety, własne stanowisko pracy itd.`, + }, + ]; } diff --git a/frontend/src/sass/_variables.scss b/frontend/src/sass/_variables.scss index e9c775e0..18aa93d2 100644 --- a/frontend/src/sass/_variables.scss +++ b/frontend/src/sass/_variables.scss @@ -1,2 +1,4 @@ /* Global variables */ -$base-font-size: 16px; +@import "~bootstrap/scss/functions"; +@import "~bootstrap/scss/variables"; +@import "~bootstrap/scss/mixins"; From c2d9a06387f38f87612f24e226e64c1fbff11b8e Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 30 May 2018 18:13:41 +0200 Subject: [PATCH 17/31] Resolving pylint issues --- backend/apps/volontulo/urls.py | 1 - backend/apps/volontulo/views/api.py | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/backend/apps/volontulo/urls.py b/backend/apps/volontulo/urls.py index 02b7b072..0e7daa38 100644 --- a/backend/apps/volontulo/urls.py +++ b/backend/apps/volontulo/urls.py @@ -105,7 +105,6 @@ # users/slug-id/contact # organizations/filter - url( r'^o/contact$', views.contact_form, diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index 4fc37a29..0d085140 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -106,7 +106,7 @@ def register_view(request): @api_view(['POST']) @authentication_classes((CsrfExemptSessionAuthentication,)) @permission_classes((AllowAny,)) -def activate_view(request, uuid): +def activate_view(_, uuid): """View responsible for activating user account.""" try: profile = UserProfile.objects.get(uuid=uuid) From c1c6de192d3a46d531ed8fa4f64d1390026d0f87 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 30 May 2018 18:18:09 +0200 Subject: [PATCH 18/31] Resolving more pylint issues --- backend/apps/volontulo/views/api.py | 1 - 1 file changed, 1 deletion(-) diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index 0d085140..b08e6034 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -102,7 +102,6 @@ def register_view(request): return Response(status=status.HTTP_201_CREATED) - @api_view(['POST']) @authentication_classes((CsrfExemptSessionAuthentication,)) @permission_classes((AllowAny,)) From 7ff4cad0f0cc43ffde826e2ffd73649ea6e97e75 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 6 Jun 2018 18:33:42 +0200 Subject: [PATCH 19/31] Corrections in backend tests --- .../tests/views/offers/test_offer_join.py | 9 +- .../apps/volontulo/tests/views/test_auth.py | 120 ------------------ backend/apps/volontulo/views/offers.py | 4 +- 3 files changed, 5 insertions(+), 128 deletions(-) diff --git a/backend/apps/volontulo/tests/views/offers/test_offer_join.py b/backend/apps/volontulo/tests/views/offers/test_offer_join.py index 56965786..16ae9493 100644 --- a/backend/apps/volontulo/tests/views/offers/test_offer_join.py +++ b/backend/apps/volontulo/tests/views/offers/test_offer_join.py @@ -162,17 +162,12 @@ def test_offers_join_valid_form_and_anonymous_user(self): response = self.client.post( '/o/offers/volontulo-offer/{}/join'.format(self.offer.id), post_data, - follow=True, ) self.assertRedirects( response, - '/o/register', + '{}/register'.format(settings.ANGULAR_ROOT), 302, - 200, - ) - self.assertContains( - response, - 'Zarejestruj się, aby zapisać się do oferty.', + fetch_redirect_response=False, ) def test_offers_join_valid_form_with_existing_email(self): diff --git a/backend/apps/volontulo/tests/views/test_auth.py b/backend/apps/volontulo/tests/views/test_auth.py index 7ffba2d7..3d25e05e 100644 --- a/backend/apps/volontulo/tests/views/test_auth.py +++ b/backend/apps/volontulo/tests/views/test_auth.py @@ -13,126 +13,6 @@ from apps.volontulo.tests import common -class TestRegister(TransactionTestCase): - """Tests for register view.""" - - def setUp(self): - """Set up each test.""" - self.client = Client() - - def test_get_method(self): - """Test for get method for view.""" - response = self.client.get('/o/register') - self.assertEqual(response.status_code, 200) - self.assertTemplateUsed(response, 'auth/login.html') - self.assertContains( - response, - 'Rejestracja w Volontulo' - ) - self.assertContains( - response, - 'Logowanie' - ) - self.assertNotIn('_auth_user_id', self.client.session) - - def test_invalid_form(self): - """Test for post method with invalid form.""" - response = self.client.post('/o/register', {}) - self.assertEqual(response.status_code, 200) - self.assertTemplateUsed(response, 'auth/login.html') - self.assertContains( - response, - 'Wprowadzono nieprawidłowy email, hasło lub nie wyrażono ' - 'zgody na przetwarzanie danych osobowych.', - ) - self.assertNotIn('_auth_user_id', self.client.session) - - def test_user_already_exists(self): - """Test for attempt of registration for user, that already exists.""" - User.objects.create_user( - 'existing@example.com', - 'existing@example.com', - '123existing' - ) - response = self.client.post('/o/register', { - 'email': 'existing@example.com', - 'password': '123existing', - 'terms_acceptance': True, - }) - self.assertEqual(response.status_code, 200) - self.assertTemplateUsed(response, 'auth/login.html') - self.assertContains( - response, - 'Użytkownik o podanym emailu już istnieje', - ) - self.assertNotIn('_auth_user_id', self.client.session) - self.assertEqual(User.objects.all().count(), 1) - - def test_register_without_term_acceptance(self): - """Test for registration without accepted terms.""" - response = self.client.post('/o/register', { - 'email': 'notacceptregistration@example.com', - 'password': '123notacceptregistration', - 'terms_acceptance': False, - }) - self.assertEqual(response.status_code, 200) - self.assertTemplateUsed(response, 'auth/login.html') - self.assertContains( - response, - 'Wprowadzono nieprawidłowy email, hasło lub nie wyrażono ' - 'zgody na przetwarzanie danych osobowych.', - ) - self.assertNotIn('_auth_user_id', self.client.session) - - def test_register_with_term_acceptance(self): - """Test for registration with accepted terms.""" - response = self.client.post('/o/register', { - 'email': 'acceptterms@example.com', - 'password': '123acceptterms', - 'terms_acceptance': True, - }) - self.assertRedirects( - response, - settings.ANGULAR_ROOT, - 302, - fetch_redirect_response=False, - ) - self.assertEqual(User.objects.all().count(), 1) - - def test_successful_registration(self): - """Test for attempt of registration for new user.""" - response = self.client.post('/o/register', { - 'email': 'new@example.com', - 'password': '123new', - 'terms_acceptance': True, - }) - self.assertRedirects( - response, - settings.ANGULAR_ROOT, - 302, - fetch_redirect_response=False, - ) - self.assertEqual(User.objects.all().count(), 1) - - def test__register_authenticated_user(self): - """Check if authenticated user can access register page.""" - # volunteer user - common.initialize_empty_volunteer() - - self.client.login( - username='volunteer1@example.com', - password='volunteer1', - ) - response = self.client.get('/o/register') - - self.assertRedirects( - response, - settings.ANGULAR_ROOT, - 302, - fetch_redirect_response=False, - ) - - class TestLogout(TestCase): """Class responsible for testing user logout view.""" diff --git a/backend/apps/volontulo/views/offers.py b/backend/apps/volontulo/views/offers.py index e3eb1888..b993d238 100644 --- a/backend/apps/volontulo/views/offers.py +++ b/backend/apps/volontulo/views/offers.py @@ -317,7 +317,9 @@ def post(request, slug, id_): # pylint: disable=unused-argument request, "Zarejestruj się, aby zapisać się do oferty." ) - return redirect('register') + return redirect('{ANGULAR_ROOT}/register'.format( + ANGULAR_ROOT=settings.ANGULAR_ROOT) + ) has_applied = Offer.objects.filter( volunteers=user, From 44d29bff705246c3370d13a1ce1fb94a88f13a22 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 6 Jun 2018 18:51:17 +0200 Subject: [PATCH 20/31] Corrections in backend tests2 --- backend/apps/volontulo/tests/views/test_auth.py | 2 -- backend/apps/volontulo/views/offers.py | 4 ++-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/backend/apps/volontulo/tests/views/test_auth.py b/backend/apps/volontulo/tests/views/test_auth.py index 3d25e05e..d847b44f 100644 --- a/backend/apps/volontulo/tests/views/test_auth.py +++ b/backend/apps/volontulo/tests/views/test_auth.py @@ -5,10 +5,8 @@ """ from django.conf import settings -from django.contrib.auth.models import User from django.test import Client from django.test import TestCase -from django.test import TransactionTestCase from apps.volontulo.tests import common diff --git a/backend/apps/volontulo/views/offers.py b/backend/apps/volontulo/views/offers.py index b993d238..d23909c0 100644 --- a/backend/apps/volontulo/views/offers.py +++ b/backend/apps/volontulo/views/offers.py @@ -318,8 +318,8 @@ def post(request, slug, id_): # pylint: disable=unused-argument "Zarejestruj się, aby zapisać się do oferty." ) return redirect('{ANGULAR_ROOT}/register'.format( - ANGULAR_ROOT=settings.ANGULAR_ROOT) - ) + ANGULAR_ROOT=settings.ANGULAR_ROOT + )) has_applied = Offer.objects.filter( volunteers=user, From 8b75e850d01adae52e5666cf5723f60113aff6b9 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 6 Jun 2018 18:57:48 +0200 Subject: [PATCH 21/31] Test removal in frontend --- .../app/register/register.component.spec.ts | 25 ------------------- 1 file changed, 25 deletions(-) delete mode 100644 frontend/src/app/register/register.component.spec.ts diff --git a/frontend/src/app/register/register.component.spec.ts b/frontend/src/app/register/register.component.spec.ts deleted file mode 100644 index 6c19551b..00000000 --- a/frontend/src/app/register/register.component.spec.ts +++ /dev/null @@ -1,25 +0,0 @@ -import { async, ComponentFixture, TestBed } from '@angular/core/testing'; - -import { RegisterComponent } from './register.component'; - -describe('RegisterComponent', () => { - let component: RegisterComponent; - let fixture: ComponentFixture; - - beforeEach(async(() => { - TestBed.configureTestingModule({ - declarations: [ RegisterComponent ] - }) - .compileComponents(); - })); - - beforeEach(() => { - fixture = TestBed.createComponent(RegisterComponent); - component = fixture.componentInstance; - fixture.detectChanges(); - }); - - it('should create', () => { - expect(component).toBeTruthy(); - }); -}); From 3a4f6b2d1dbea88cb27b6e1486024ad8ee218498 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 6 Jun 2018 19:04:53 +0200 Subject: [PATCH 22/31] Removed activation.component.spec.ts --- .../activation/activation.component.spec.ts | 25 ------------------- 1 file changed, 25 deletions(-) delete mode 100644 frontend/src/app/activation/activation.component.spec.ts diff --git a/frontend/src/app/activation/activation.component.spec.ts b/frontend/src/app/activation/activation.component.spec.ts deleted file mode 100644 index 5d082e9c..00000000 --- a/frontend/src/app/activation/activation.component.spec.ts +++ /dev/null @@ -1,25 +0,0 @@ -import { async, ComponentFixture, TestBed } from '@angular/core/testing'; - -import { ActivationComponent } from './activation.component'; - -describe('ActivationComponent', () => { - let component: ActivationComponent; - let fixture: ComponentFixture; - - beforeEach(async(() => { - TestBed.configureTestingModule({ - declarations: [ ActivationComponent ] - }) - .compileComponents(); - })); - - beforeEach(() => { - fixture = TestBed.createComponent(ActivationComponent); - component = fixture.componentInstance; - fixture.detectChanges(); - }); - - it('should create', () => { - expect(component).toBeTruthy(); - }); -}); From 534f410e49faadaef92ef640a2e661935597e1c9 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 20 Jun 2018 20:56:45 +0200 Subject: [PATCH 23/31] RegisterComponentUpdate --- backend/apps/volontulo/views/api.py | 13 ++++++------- .../src/app/register/register.component.html | 17 ++++++----------- frontend/src/app/register/register.component.ts | 7 +++++++ 3 files changed, 19 insertions(+), 18 deletions(-) diff --git a/backend/apps/volontulo/views/api.py b/backend/apps/volontulo/views/api.py index 09b27a69..0099412e 100644 --- a/backend/apps/volontulo/views/api.py +++ b/backend/apps/volontulo/views/api.py @@ -82,14 +82,13 @@ def register_view(request): email = request.data.get('email') password = request.data.get('password') - - user = User.objects.create_user( - username=email, - email=email, - password=password, - is_active=False, - ) try: + user = User.objects.create_user( + username=email, + email=email, + password=password, + is_active=False, + ) user.save() except IntegrityError: return Response(status=status.HTTP_201_CREATED) diff --git a/frontend/src/app/register/register.component.html b/frontend/src/app/register/register.component.html index 076e0d0b..d8d5f093 100644 --- a/frontend/src/app/register/register.component.html +++ b/frontend/src/app/register/register.component.html @@ -20,23 +20,18 @@

      Rejestracja w Volontulo

      - Pole 'Hasło' jest wymagane. -
      - + -
      - -
      - Wymagana zgoda na przetwarzanie danych osobowych. -
      - -
      +
      +
      + Wymagana zgoda na przetwarzanie danych osobowych.
      - + diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index 0d20ff80..f8847fab 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -17,12 +17,19 @@ export class RegisterComponent { ACCEPT_TERMS = 'Wyrażam zgodę na przetwarzanie moich danych osobowych'; registrationSuccessful = false; userIsAuthenticated = false; + termsAccepted = false; + showTermsWarning = false; constructor(private authService: AuthService, ) { } register(): void { + if (!this.termsAccepted) { + this.showTermsWarning = true; + return; + } + this.registrationSuccessful = false; this.userIsAuthenticated = false; this.authService.register(this.registerModel.email, this.registerModel.password) From 08cef2acd8feed30c82c3c15abdcf4fe2834d53c Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 27 Jun 2018 18:17:42 +0200 Subject: [PATCH 24/31] RegisterComponentIssues --- .../src/app/register/register.component.html | 52 ++++++++++++++----- .../src/app/register/register.component.ts | 20 +++---- 2 files changed, 50 insertions(+), 22 deletions(-) diff --git a/frontend/src/app/register/register.component.html b/frontend/src/app/register/register.component.html index d8d5f093..9fd94fe8 100644 --- a/frontend/src/app/register/register.component.html +++ b/frontend/src/app/register/register.component.html @@ -2,11 +2,18 @@

      Rejestracja w Volontulo

      Email:

      - +
      -
      Pole 'Email' jest wymagane.
      @@ -17,21 +24,42 @@

      Rejestracja w Volontulo

      Hasło:

      - +
      Pole 'Hasło' jest wymagane.
      - +
      -
      +
      Wymagana zgoda na przetwarzanie danych osobowych.
      - - - + + + diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index f8847fab..89d4223b 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -1,9 +1,10 @@ -import { Component } from '@angular/core'; +import { Component, ViewChild } from '@angular/core'; import { RegisterRequestModel } from 'app/auth.models'; import { AuthService } from '../auth.service'; import { Observable } from 'rxjs/Observable'; import { RedirectComponent } from 'app/redirect.component'; import { Params } from '@angular/router'; +import { NgControl } from '@angular/forms'; @Component({ selector: 'volontulo-register', @@ -17,18 +18,17 @@ export class RegisterComponent { ACCEPT_TERMS = 'Wyrażam zgodę na przetwarzanie moich danych osobowych'; registrationSuccessful = false; userIsAuthenticated = false; - termsAccepted = false; - showTermsWarning = false; + + @ViewChild('checkboxTA') public checkboxTA: NgControl; constructor(private authService: AuthService, ) { } register(): void { - if (!this.termsAccepted) { - this.showTermsWarning = true; + this.checkboxTA.control.markAsDirty(); + if (!this.checkboxTA.control.value) return; - } this.registrationSuccessful = false; this.userIsAuthenticated = false; @@ -39,11 +39,11 @@ export class RegisterComponent { } return Observable.of(null); }, - err => { - if (err.status === 400) { - this.userIsAuthenticated = true; + err => { + if (err.status === 400) { + this.userIsAuthenticated = true; + } } - } ); } } From ea9aba5bacd73631a9e8ca156fe73f78954136a9 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 27 Jun 2018 18:48:54 +0200 Subject: [PATCH 25/31] RegisterComponentIssues2 --- frontend/src/app/register/register.component.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index 89d4223b..16cfa144 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -27,8 +27,9 @@ export class RegisterComponent { register(): void { this.checkboxTA.control.markAsDirty(); - if (!this.checkboxTA.control.value) + if (!this.checkboxTA.control.value){ return; + } this.registrationSuccessful = false; this.userIsAuthenticated = false; From aca68d08bcdff5ccd0cd5294801f157708d4b395 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 27 Jun 2018 18:54:08 +0200 Subject: [PATCH 26/31] RegisterComponentIssues3 --- frontend/src/app/register/register.component.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index 16cfa144..c7158e7d 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -27,7 +27,7 @@ export class RegisterComponent { register(): void { this.checkboxTA.control.markAsDirty(); - if (!this.checkboxTA.control.value){ + if (!this.checkboxTA.control.value) { return; } From 880257f511bf0bd0e44ef619472b49549caf0295 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 11 Jul 2018 18:13:50 +0200 Subject: [PATCH 27/31] RegisterComponentIssues4 --- frontend/src/app/register/register.component.html | 1 - frontend/src/app/register/register.component.ts | 8 ++++---- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/frontend/src/app/register/register.component.html b/frontend/src/app/register/register.component.html index 9fd94fe8..bea5861d 100644 --- a/frontend/src/app/register/register.component.html +++ b/frontend/src/app/register/register.component.html @@ -11,7 +11,6 @@

      Rejestracja w Volontulo

      required email maxlength="150" - pattern="^\w+([\.-]?\w+)*@\w+([\.-]?\w+)*(\.\w{2,})+$" #email="ngModel" />
      diff --git a/frontend/src/app/register/register.component.ts b/frontend/src/app/register/register.component.ts index c7158e7d..8318d7e3 100644 --- a/frontend/src/app/register/register.component.ts +++ b/frontend/src/app/register/register.component.ts @@ -40,11 +40,11 @@ export class RegisterComponent { } return Observable.of(null); }, - err => { - if (err.status === 400) { - this.userIsAuthenticated = true; - } + err => { + if (err.status === 400) { + this.userIsAuthenticated = true; } + } ); } } From a2790b4c58f004796b3bc7eb2b4375aa149c5264 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 3 Oct 2018 18:27:36 +0200 Subject: [PATCH 28/31] [986] component showing offers that user joined --- .../src/app/account/account.component.html | 3 ++- frontend/src/app/account/account.component.ts | 1 + frontend/src/app/app.module.ts | 4 ++- .../homepage-offer.component.html | 4 +-- .../src/app/homepage-offer/offers.service.ts | 4 +++ .../offers-box/offers-box.component.html | 11 ++++++++ .../offers-box/offers-box.component.scss | 0 .../offers-box/offers-box.component.spec.ts | 25 +++++++++++++++++++ .../offers/offers-box/offers-box.component.ts | 21 ++++++++++++++++ 9 files changed, 69 insertions(+), 4 deletions(-) create mode 100644 frontend/src/app/offers/offers-box/offers-box.component.html create mode 100644 frontend/src/app/offers/offers-box/offers-box.component.scss create mode 100644 frontend/src/app/offers/offers-box/offers-box.component.spec.ts create mode 100644 frontend/src/app/offers/offers-box/offers-box.component.ts diff --git a/frontend/src/app/account/account.component.html b/frontend/src/app/account/account.component.html index c70e4123..dc102cdd 100644 --- a/frontend/src/app/account/account.component.html +++ b/frontend/src/app/account/account.component.html @@ -14,7 +14,8 @@

      Profil

      Aktywność

      -

      Zgłoś się w jednej z dostępnych ofert wolontariatu i zapełnij to miejsce.

      + +
      diff --git a/frontend/src/app/account/account.component.ts b/frontend/src/app/account/account.component.ts index d4b0be53..5d2dba1d 100644 --- a/frontend/src/app/account/account.component.ts +++ b/frontend/src/app/account/account.component.ts @@ -5,6 +5,7 @@ import { map } from 'rxjs/operators'; import { AuthService } from '../auth.service'; import { Organization } from '../organization/organization.model'; +import { OffersBoxComponent } from '../offers/offers-box/offers-box.component'; @Component({ selector: 'volontulo-account', diff --git a/frontend/src/app/app.module.ts b/frontend/src/app/app.module.ts index fd3391b6..9992fbdb 100644 --- a/frontend/src/app/app.module.ts +++ b/frontend/src/app/app.module.ts @@ -55,6 +55,7 @@ import { FormErrorComponent } from './form-error/form-error.component'; import { ContactService } from './contact.service'; import { UserProfileComponent } from './user-profile/user-profile.component'; import { FontAwesomeModule } from '@fortawesome/angular-fontawesome'; +import { OffersBoxComponent } from './offers/offers-box/offers-box.component'; Raven.config(environment.sentryDSN).install(); @@ -212,7 +213,8 @@ registerLocaleData(localePl); ContactComponent, FormErrorComponent, OrganizationsListComponent, - UserProfileComponent + UserProfileComponent, + OffersBoxComponent ], imports: [ BrowserModule.withServerTransition({ appId: 'volontulo' }), diff --git a/frontend/src/app/homepage-offer/homepage-offer.component.html b/frontend/src/app/homepage-offer/homepage-offer.component.html index 11c6de8a..b4624c35 100644 --- a/frontend/src/app/homepage-offer/homepage-offer.component.html +++ b/frontend/src/app/homepage-offer/homepage-offer.component.html @@ -5,7 +5,7 @@

      - {{ offer.title }} + {{ offer.title }}

      diff --git a/frontend/src/app/homepage-offer/offers.service.ts b/frontend/src/app/homepage-offer/offers.service.ts index 5a506a6d..618e1370 100644 --- a/frontend/src/app/homepage-offer/offers.service.ts +++ b/frontend/src/app/homepage-offer/offers.service.ts @@ -35,4 +35,8 @@ export class OffersService { return this.http.put(`${environment.apiRoot}/offers/${id}/`, offer); } + getUserOffers(): Observable { + return this.http.get(`${environment.apiRoot}/joined-offers/`) + .map(offers => offers.map(offer => loadDefaultImage(offer))); + } } diff --git a/frontend/src/app/offers/offers-box/offers-box.component.html b/frontend/src/app/offers/offers-box/offers-box.component.html new file mode 100644 index 00000000..02e8622f --- /dev/null +++ b/frontend/src/app/offers/offers-box/offers-box.component.html @@ -0,0 +1,11 @@ +
      +

      Zgłoś się w jednej z dostępnych ofert wolontariatu i zapełnij to miejsce.

      +
      + + +
      +
      + +
      +
      +       diff --git a/frontend/src/app/offers/offers-box/offers-box.component.scss b/frontend/src/app/offers/offers-box/offers-box.component.scss new file mode 100644 index 00000000..e69de29b diff --git a/frontend/src/app/offers/offers-box/offers-box.component.spec.ts b/frontend/src/app/offers/offers-box/offers-box.component.spec.ts new file mode 100644 index 00000000..d1cd8c5c --- /dev/null +++ b/frontend/src/app/offers/offers-box/offers-box.component.spec.ts @@ -0,0 +1,25 @@ +import { async, ComponentFixture, TestBed } from '@angular/core/testing'; + +import { OffersBoxComponent } from './offers-box.component'; + +describe('OffersBoxComponent', () => { + let component: OffersBoxComponent; + let fixture: ComponentFixture; + + beforeEach(async(() => { + TestBed.configureTestingModule({ + declarations: [ OffersBoxComponent ] + }) + .compileComponents(); + })); + + beforeEach(() => { + fixture = TestBed.createComponent(OffersBoxComponent); + component = fixture.componentInstance; + fixture.detectChanges(); + }); + + it('should create', () => { + expect(component).toBeTruthy(); + }); +}); diff --git a/frontend/src/app/offers/offers-box/offers-box.component.ts b/frontend/src/app/offers/offers-box/offers-box.component.ts new file mode 100644 index 00000000..ca46eb55 --- /dev/null +++ b/frontend/src/app/offers/offers-box/offers-box.component.ts @@ -0,0 +1,21 @@ +import { Component, OnInit } from '@angular/core'; +import { OffersService } from '../../homepage-offer/offers.service'; +import { Offer } from '../../homepage-offer/offers.model' + +@Component({ + selector: 'volontulo-offers-box', + templateUrl: './offers-box.component.html', + styleUrls: ['./offers-box.component.scss'], + providers: [OffersService] +}) +export class OffersBoxComponent implements OnInit { + offers: Offer[] = []; + + constructor(private offersService: OffersService) { + } + + ngOnInit() { + this.offersService.getUserOffers() + .subscribe(offers => this.offers = offers); + } +} From 37f90e241e8121fa31dbf1911824a491bee037a6 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Wed, 3 Oct 2018 20:03:20 +0200 Subject: [PATCH 29/31] Implemented observable use in offersbox component --- frontend/src/app/account/account.component.html | 1 - frontend/src/app/account/account.component.ts | 1 - .../src/app/offers/offers-box/offers-box.component.html | 4 ++-- .../src/app/offers/offers-box/offers-box.component.scss | 0 .../src/app/offers/offers-box/offers-box.component.ts | 8 +++----- 5 files changed, 5 insertions(+), 9 deletions(-) delete mode 100644 frontend/src/app/offers/offers-box/offers-box.component.scss diff --git a/frontend/src/app/account/account.component.html b/frontend/src/app/account/account.component.html index dc102cdd..9c49c1b2 100644 --- a/frontend/src/app/account/account.component.html +++ b/frontend/src/app/account/account.component.html @@ -14,7 +14,6 @@

      Profil

      Aktywność

      -
      diff --git a/frontend/src/app/account/account.component.ts b/frontend/src/app/account/account.component.ts index 5d2dba1d..d4b0be53 100644 --- a/frontend/src/app/account/account.component.ts +++ b/frontend/src/app/account/account.component.ts @@ -5,7 +5,6 @@ import { map } from 'rxjs/operators'; import { AuthService } from '../auth.service'; import { Organization } from '../organization/organization.model'; -import { OffersBoxComponent } from '../offers/offers-box/offers-box.component'; @Component({ selector: 'volontulo-account', diff --git a/frontend/src/app/offers/offers-box/offers-box.component.html b/frontend/src/app/offers/offers-box/offers-box.component.html index 02e8622f..fbf01195 100644 --- a/frontend/src/app/offers/offers-box/offers-box.component.html +++ b/frontend/src/app/offers/offers-box/offers-box.component.html @@ -1,10 +1,10 @@ -
      +

      Zgłoś się w jednej z dostępnych ofert wolontariatu i zapełnij to miejsce.

      -
      +
      diff --git a/frontend/src/app/offers/offers-box/offers-box.component.scss b/frontend/src/app/offers/offers-box/offers-box.component.scss deleted file mode 100644 index e69de29b..00000000 diff --git a/frontend/src/app/offers/offers-box/offers-box.component.ts b/frontend/src/app/offers/offers-box/offers-box.component.ts index ca46eb55..e5ee19d0 100644 --- a/frontend/src/app/offers/offers-box/offers-box.component.ts +++ b/frontend/src/app/offers/offers-box/offers-box.component.ts @@ -1,21 +1,19 @@ import { Component, OnInit } from '@angular/core'; import { OffersService } from '../../homepage-offer/offers.service'; import { Offer } from '../../homepage-offer/offers.model' +import { Observable } from 'rxjs/Observable'; @Component({ selector: 'volontulo-offers-box', templateUrl: './offers-box.component.html', - styleUrls: ['./offers-box.component.scss'], providers: [OffersService] }) export class OffersBoxComponent implements OnInit { - offers: Offer[] = []; - + public offers$: Observable; constructor(private offersService: OffersService) { } ngOnInit() { - this.offersService.getUserOffers() - .subscribe(offers => this.offers = offers); + this.offers$ = this.offersService.getUserOffers(); } } From b60a51ada48a4e3948f52f9f8287a205d85276e6 Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Sat, 27 Oct 2018 22:38:50 +0200 Subject: [PATCH 30/31] ResolvedTravisIssues --- .../offers-box/offers-box.component.html | 18 ++++++++++-------- .../offers-box/offers-box.component.spec.ts | 17 +++++++++++++++-- .../offers/offers-box/offers-box.component.ts | 4 ++-- 3 files changed, 27 insertions(+), 12 deletions(-) diff --git a/frontend/src/app/offers/offers-box/offers-box.component.html b/frontend/src/app/offers/offers-box/offers-box.component.html index fbf01195..2efa9787 100644 --- a/frontend/src/app/offers/offers-box/offers-box.component.html +++ b/frontend/src/app/offers/offers-box/offers-box.component.html @@ -1,11 +1,13 @@ -
      -

      Zgłoś się w jednej z dostępnych ofert wolontariatu i zapełnij to miejsce.

      -
      - - -
      -
      - +
      +
      +
      +
      + +
      +
      + + +

      Zgłoś się w jednej z dostępnych ofert wolontariatu i zapełnij to miejsce.

       diff --git a/frontend/src/app/offers/offers-box/offers-box.component.spec.ts b/frontend/src/app/offers/offers-box/offers-box.component.spec.ts index d1cd8c5c..fb81541e 100644 --- a/frontend/src/app/offers/offers-box/offers-box.component.spec.ts +++ b/frontend/src/app/offers/offers-box/offers-box.component.spec.ts @@ -1,14 +1,26 @@ import { async, ComponentFixture, TestBed } from '@angular/core/testing'; +import { HttpClient } from '@angular/common/http'; +import { HttpClientTestingModule } from '@angular/common/http/testing'; +import { RouterTestingModule } from '@angular/router/testing'; +import { HomepageOfferComponent } from '../../homepage-offer/homepage-offer.component'; import { OffersBoxComponent } from './offers-box.component'; +import { OffersService } from '../../homepage-offer/offers.service'; describe('OffersBoxComponent', () => { let component: OffersBoxComponent; let fixture: ComponentFixture; + let offersService: OffersService; + let httpClient: HttpClient; beforeEach(async(() => { TestBed.configureTestingModule({ - declarations: [ OffersBoxComponent ] + imports: [RouterTestingModule, HttpClientTestingModule], + declarations: [ + OffersBoxComponent, + HomepageOfferComponent + ], + providers: [OffersService] }) .compileComponents(); })); @@ -16,7 +28,8 @@ describe('OffersBoxComponent', () => { beforeEach(() => { fixture = TestBed.createComponent(OffersBoxComponent); component = fixture.componentInstance; - fixture.detectChanges(); + offersService = TestBed.get(OffersService); + httpClient = TestBed.get(HttpClient); }); it('should create', () => { diff --git a/frontend/src/app/offers/offers-box/offers-box.component.ts b/frontend/src/app/offers/offers-box/offers-box.component.ts index e5ee19d0..b1dd8bc0 100644 --- a/frontend/src/app/offers/offers-box/offers-box.component.ts +++ b/frontend/src/app/offers/offers-box/offers-box.component.ts @@ -10,8 +10,8 @@ import { Observable } from 'rxjs/Observable'; }) export class OffersBoxComponent implements OnInit { public offers$: Observable; - constructor(private offersService: OffersService) { - } + + constructor(private offersService: OffersService) { } ngOnInit() { this.offers$ = this.offersService.getUserOffers(); From e50b3e8eb8ae8e2bda7d9d00edf2f8771f73470b Mon Sep 17 00:00:00 2001 From: jacekkalbarczyk Date: Mon, 29 Oct 2018 23:10:30 +0100 Subject: [PATCH 31/31] MergingMasterUpdates --- frontend/src/app/app.module.ts | 2 +- .../offers/offers-box/offers-box.component.html | 0 .../offers/offers-box/offers-box.component.spec.ts | 6 +++--- .../offers/offers-box/offers-box.component.ts | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) rename frontend/src/app/{ => components}/offers/offers-box/offers-box.component.html (100%) rename frontend/src/app/{ => components}/offers/offers-box/offers-box.component.spec.ts (80%) rename frontend/src/app/{ => components}/offers/offers-box/offers-box.component.ts (77%) diff --git a/frontend/src/app/app.module.ts b/frontend/src/app/app.module.ts index 9c4e2528..33c728fc 100644 --- a/frontend/src/app/app.module.ts +++ b/frontend/src/app/app.module.ts @@ -9,7 +9,6 @@ import { NgbModule } from '@ng-bootstrap/ng-bootstrap'; import { CookieModule } from 'ngx-cookie'; import * as Raven from 'raven-js'; import { FontAwesomeModule } from '@fortawesome/angular-fontawesome'; -import { OffersBoxComponent } from './offers/offers-box/offers-box.component'; import { IMaskModule } from 'angular-imask'; import { AppComponent } from 'app/app.component'; @@ -26,6 +25,7 @@ import { HomepageOfferComponent } from 'app/components/homepage-offer/homepage-o import { LoginComponent } from 'app/components/login/login.component'; import { MessagesComponent } from 'app/components/messages/messages.component'; import { CreateOfferComponent } from 'app/components/offers/create-offer/create-offer.component'; +import { OffersBoxComponent } from 'app/components/offers/offers-box/offers-box.component'; import { OfferDetailComponent } from 'app/components/offers/offer-detail/offer-detail.component'; import { OrganizationComponent } from 'app/components/organization/organization.component'; import { OrganizationContactComponent } from 'app/components/organization/organization-contact/organization-contact.component'; diff --git a/frontend/src/app/offers/offers-box/offers-box.component.html b/frontend/src/app/components/offers/offers-box/offers-box.component.html similarity index 100% rename from frontend/src/app/offers/offers-box/offers-box.component.html rename to frontend/src/app/components/offers/offers-box/offers-box.component.html diff --git a/frontend/src/app/offers/offers-box/offers-box.component.spec.ts b/frontend/src/app/components/offers/offers-box/offers-box.component.spec.ts similarity index 80% rename from frontend/src/app/offers/offers-box/offers-box.component.spec.ts rename to frontend/src/app/components/offers/offers-box/offers-box.component.spec.ts index fb81541e..6c50d89f 100644 --- a/frontend/src/app/offers/offers-box/offers-box.component.spec.ts +++ b/frontend/src/app/components/offers/offers-box/offers-box.component.spec.ts @@ -3,9 +3,9 @@ import { HttpClient } from '@angular/common/http'; import { HttpClientTestingModule } from '@angular/common/http/testing'; import { RouterTestingModule } from '@angular/router/testing'; -import { HomepageOfferComponent } from '../../homepage-offer/homepage-offer.component'; -import { OffersBoxComponent } from './offers-box.component'; -import { OffersService } from '../../homepage-offer/offers.service'; +import { HomepageOfferComponent } from 'app/components/homepage-offer/homepage-offer.component'; +import { OffersBoxComponent } from 'app/components/offers/offers-box/offers-box.component'; +import { OffersService } from 'app/services/offers.service'; describe('OffersBoxComponent', () => { let component: OffersBoxComponent; diff --git a/frontend/src/app/offers/offers-box/offers-box.component.ts b/frontend/src/app/components/offers/offers-box/offers-box.component.ts similarity index 77% rename from frontend/src/app/offers/offers-box/offers-box.component.ts rename to frontend/src/app/components/offers/offers-box/offers-box.component.ts index b1dd8bc0..b88e3ed3 100644 --- a/frontend/src/app/offers/offers-box/offers-box.component.ts +++ b/frontend/src/app/components/offers/offers-box/offers-box.component.ts @@ -1,6 +1,6 @@ import { Component, OnInit } from '@angular/core'; -import { OffersService } from '../../homepage-offer/offers.service'; -import { Offer } from '../../homepage-offer/offers.model' +import { OffersService } from 'app/services/offers.service'; +import { Offer } from 'app/models/offer.model'; import { Observable } from 'rxjs/Observable'; @Component({