Skip to content

perf: added teak support#16

Merged
Asespinel merged 5 commits into
masterfrom
ase/added-teak-support
Jun 23, 2025
Merged

perf: added teak support#16
Asespinel merged 5 commits into
masterfrom
ase/added-teak-support

Conversation

@Asespinel

Copy link
Copy Markdown
Contributor

Description

This PR aims to support the Teak release.

Testing instructions

1, Create a Teak environment using Tutor or TVM.
2. Install webhook-xblock using this branch you can follow the official tutor documentation.
3. In a course of study:
Navigate to Settings -> Advanced Settings and go to Advanced Module List settings.
Add: "webhook-xblock"
4. Add the component under the "advanced" section
5.
image

@Asespinel Asespinel requested a review from a team as a code owner June 22, 2025 21:38
@Asespinel Asespinel force-pushed the ase/added-teak-support branch from 0e0e243 to e655dc3 Compare June 22, 2025 22:29

@magajh magajh left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Asespinel have we checked that all the gh workflow dependencies are up to date?


Also, we still need to update the changelog. Apart from updating it with information about this new release, can we please fix the changelog format?
It looks like the first entry ended up at the end of the file instead of at the beginning
https://github.com/eduNEXT/webhook-xblock/blob/master/CHANGELOG.rst#L49-L52

Comment thread webhook_xblock/__init__.py Outdated
from .webhook_xblock import WebhookXblock

__version__ = '1.0.1'
__version__ = '2.0.0'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This should be a minor bump, there are no breaking changes in this PR to make it a major

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I changed it to a minor bump

Comment thread README.rst Outdated
+------------------+---------------+
| Sumac | >= 1.0.0 |
+------------------+---------------+
| Teak | >= 1.0.0 |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Asespinel we need to update this version

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I fixed it.

Comment thread CHANGELOG.rst Outdated
Changed
=======

* Updated dependency versions in requirements

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's please improve this message before merging

@Asespinel Asespinel merged commit 6156134 into master Jun 23, 2025
3 checks passed
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.

2 participants