Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edit Profile Form #9811

Merged
merged 5 commits into from
Nov 18, 2024
Merged

Conversation

danieljames-dj
Copy link
Member

@danieljames-dj danieljames-dj commented Aug 24, 2024

Currently the edit profile form is merged along with the main contact form, but this is making the main contact form increasingly complex. This PR is an attempt to make 'edit profile' portion of the form into it's own separate form.

Temporarily, this particular form is not linked anywhere else in website, this can be accessed only if people know the URL. Once this is merged, I'll raise another PR to remove everything related to 'edit profile' from main contact form and instead if the user selects 'edit profile', then will give the user the button to redirect to 'edit profile form' page.

EDIT: This is now linked with the main contact form.

@danieljames-dj danieljames-dj force-pushed the edit-profile-form branch 2 times, most recently from 47d7e5d to 317ca23 Compare August 24, 2024 15:20
@danieljames-dj
Copy link
Member Author

@thewca-bot deploy staging

2 similar comments
@danieljames-dj
Copy link
Member Author

@thewca-bot deploy staging

@maxidragon
Copy link
Member

@thewca-bot deploy staging

@maxidragon
Copy link
Member

(If anyone is wondering why I tried to trigger the deployment, Daniel allowed me to try to check if everyone has access to trigger it)

app/controllers/contacts_controller.rb Outdated Show resolved Hide resolved
app/controllers/contacts_controller.rb Outdated Show resolved Hide resolved
app/controllers/contacts_controller.rb Outdated Show resolved Hide resolved
app/controllers/contacts_controller.rb Outdated Show resolved Hide resolved
app/controllers/contacts_controller.rb Show resolved Hide resolved
@danieljames-dj danieljames-dj force-pushed the edit-profile-form branch 2 times, most recently from f7acc4a to e3ebff2 Compare November 3, 2024 17:12
@danieljames-dj danieljames-dj merged commit 4c00e1c into thewca:main Nov 18, 2024
2 checks passed
@danieljames-dj danieljames-dj deleted the edit-profile-form branch November 18, 2024 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants