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

Remove project location #1565

Closed
5 of 6 tasks
JackHaeg opened this issue Jan 30, 2024 · 0 comments · Fixed by #1567
Closed
5 of 6 tasks

Remove project location #1565

JackHaeg opened this issue Jan 30, 2024 · 0 comments · Fixed by #1567
Assignees
Labels
p-feature: Project Details ready for dev lead Add to issue when dev lead needs to take some action role: Front End size: 1pt Can be done in 4-6 hours

Comments

@JackHaeg
Copy link
Member

JackHaeg commented Jan 30, 2024

Overview

Currently, users are required to provide a project's location / Zoom link on the "Project Management" screen. This information is no longer required, given that specific locations / Zoom links are input for each event (via the "Recurring Events" feature).

Remove the "Location" field and selection from the "Project Management" screen.

Action Items

  • On the "Project Management" screen, remove the title "Location",
  • Remove the radio buttons "In-Person" and "Remote"
  • Remove the input field
  • Remove all logic requiring the Location field to be complete (and include a valid zoom link if user selects "Remote") in order to save changes to a project
  • Test to ensure that edits to all other data is saved

Additiional Info

  • Comment out the location from the client/src/components/data.js file

Resources/Instructions

  • See below
Screenshot 2024-01-29 at 8 39 41 PM
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p-feature: Project Details ready for dev lead Add to issue when dev lead needs to take some action role: Front End size: 1pt Can be done in 4-6 hours
Projects
Status: Final QA (product)
Development

Successfully merging a pull request may close this issue.

3 participants