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

Lookyloo integration #41

Merged
merged 2 commits into from
Nov 25, 2024
Merged

Lookyloo integration #41

merged 2 commits into from
Nov 25, 2024

Conversation

romainkieffer
Copy link
Contributor

Generic requirements in order to contribute:

  • One Pull Request per fix/feature/change/...
  • Keep the amount of commits per PR as small as possible: if for any reason, you need to fix your commit after the pull request, please squash the changes in one single commit (or tell us why not)
  • Always make sure it is mergeable in the default branch
  • Any major changes adding a functionality should be disabled by default in the configuration

What does it do?

If it fixes an existing issue, please use GitHub syntax: #<IssueID>

Questions

  • Does it require a DB change?
  • Are you using it in production?

Release Type:

  • Major
  • Minor
  • Patch

@pparage pparage self-requested a review November 25, 2024 08:49
@pparage pparage added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 25, 2024
@romainkieffer romainkieffer changed the title Dev Lookyloo integration Nov 25, 2024
@romainkieffer romainkieffer merged commit de798fa into main Nov 25, 2024
2 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants