Skip to content

Conversation

@rafaeljusto
Copy link
Contributor

Description

Allow handling skills.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Testing

  • Tests pass locally (go test -v ./...)
  • Added/updated tests for new functionality

Checklist

  • Code follows project style guidelines
  • Self-reviewed the code
  • Added necessary documentation
  • No new warnings or errors

Allow handling skills.
@rafaeljusto rafaeljusto requested a review from a team as a code owner December 29, 2025 13:57
@rafaeljusto rafaeljusto merged commit bc6c5cb into main Dec 29, 2025
3 checks passed
@rafaeljusto rafaeljusto deleted the feature/skills branch December 29, 2025 13:59

// SkillDelete deletes a skill in Teamwork.com.
func SkillDelete(engine *twapi.Engine) toolsets.ToolWrapper {
return toolsets.ToolWrapper{

Choose a reason for hiding this comment

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

Just wondering @rafaeljusto is the underlying endpoint for deleting a skill a publicly documented endpoint currently? I ask because i'm thinking about reports, and how deleting a skill might skew reporting?

Choose a reason for hiding this comment

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

Also great to see your recents additions here btw! Nice one!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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.

3 participants