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

Update fields for Event Tasks #2094

Open
maneesha opened this issue Nov 3, 2021 · 2 comments
Open

Update fields for Event Tasks #2094

maneesha opened this issue Nov 3, 2021 · 2 comments
Labels
Getting started Good for new contributors to get started

Comments

@maneesha
Copy link
Contributor

maneesha commented Nov 3, 2021

Right now, for Event Tasks we have four fields - Person, Role, Title, URL.
We have never used the Title or URL field so these should be removed at least from the UI.
I will leave it to @pbanaszkiewicz to decide if there is a reason to keep them in the back end database model.

@maneesha
Copy link
Contributor Author

@pbanaszkiewicz I can start working on this PR if you let me know - should we change just the UI, or should we remove these fields from the model as well?

@pbanaszkiewicz
Copy link
Contributor

@maneesha I don't think title or URL are used. I think you can remove them from both UI and database.

Once you remove the fields and apply the migration, you should run the tests to figure out if something broke.

Finally the database documentation should be updated to reflect removal of these fields.

@elichad elichad added the Getting started Good for new contributors to get started label Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Getting started Good for new contributors to get started
Projects
None yet
Development

No branches or pull requests

3 participants