diff --git a/src/registrar/forms/portfolio.py b/src/registrar/forms/portfolio.py index 4bc7ec046..ab3b3a269 100644 --- a/src/registrar/forms/portfolio.py +++ b/src/registrar/forms/portfolio.py @@ -135,10 +135,6 @@ class NewMemberForm(forms.ModelForm): model = User fields = ['email'] #, 'grade', 'sport'] - def __init__(self, *args, **kwargs): - super(NewMemberForm, self).__init__(*args, **kwargs) - # self.fields['sport'].choices = [] - def clean(self): cleaned_data = super().clean() diff --git a/src/registrar/templates/portfolio_members_add_new.html b/src/registrar/templates/portfolio_members_add_new.html index 97e92b560..29f6fc2d7 100644 --- a/src/registrar/templates/portfolio_members_add_new.html +++ b/src/registrar/templates/portfolio_members_add_new.html @@ -9,10 +9,13 @@ {% endblock %} {% block portfolio_content %} + + {% block messages %} {% include "includes/form_messages.html" %} {% endblock messages%} + + {% block new_member_header %}

Add a new member

{% endblock new_member_header %} {% include "includes/required_fields.html" %} -{% block form_fields %} -
+

Email

@@ -45,12 +48,12 @@
+

Member Access

- Select the level of access for this member. *
@@ -70,8 +73,7 @@
- - +

Admin access permissions

@@ -89,7 +91,7 @@ {% endwith %}
- +

Basic member permissions

@@ -99,6 +101,7 @@
+
-
-{% endblock form_fields%} + {% endblock portfolio_content%} diff --git a/src/registrar/views/portfolios.py b/src/registrar/views/portfolios.py index 49925b2ef..0674080d1 100644 --- a/src/registrar/views/portfolios.py +++ b/src/registrar/views/portfolios.py @@ -233,6 +233,32 @@ class NewMemberView(PortfolioMembersPermissionView, FormMixin): self.object = self.get_object() form = self.get_form() return self.render_to_response(self.get_context_data(form=form)) + + def post(self, request, *args, **kwargs): + """Handle POST requests to process form submission.""" + self.object = self.get_object() + form = self.get_form() + if form.is_valid(): + return self.form_valid(form) + else: + return self.form_invalid(form) + + def form_valid(self, form): + """Handle the case when the form is valid.""" + # self.object = form.save(commit=False) + # self.object.creator = self.request.user + # self.object.save() + # messages.success(self.request, "The organization information for this portfolio has been updated.") + return super().form_valid(form) + + def form_invalid(self, form): + """Handle the case when the form is invalid.""" + return self.render_to_response(self.get_context_data(form=form)) + + def get_success_url(self): + """Redirect to the overview page for the portfolio.""" + return reverse("members") + ########################################## # TODO: future ticket