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

Added ability to remove activities and small test for it #69

Merged
merged 3 commits into from
Dec 19, 2022

Conversation

gomezzz
Copy link
Collaborator

@gomezzz gomezzz commented Dec 12, 2022

Description

as title

Resolved Issues

How Has This Been Tested?

  • New test

Related Pull Requests

N/A

@gomezzz gomezzz added enhancement New feature or request tests Anything related to the tests user-facing Anything that users can interact with labels Dec 12, 2022
Copy link
Collaborator

@GabrieleMeoni GabrieleMeoni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It works fine for me. Just update the missing docstring and then it is fine for me.

paseos/activities/activity_manager.py Outdated Show resolved Hide resolved
paseos/paseos.py Outdated Show resolved Hide resolved
@gomezzz gomezzz merged commit b9e1147 into main Dec 19, 2022
@gomezzz gomezzz deleted the Adding-a-function-to-delete-an-activity branch December 19, 2022 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tests Anything related to the tests user-facing Anything that users can interact with
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding a function to delete an activity in PASEOS
2 participants