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

BUG: Changed limit_by_projects to from_projects for server datetime search action #287

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

Neonbluestoplight
Copy link
Contributor

Description:

limit_by_projects was returning:

TypeError: parse_meta_params() got an unexpected keyword argument 'limit_by_projects'

Changed to from_projects.


Submitter:

Have you (where applicable):

  • Added unit tests?
  • Checked the latest commit runs on Dev?
  • Updated the example config file(s) and README?

Reviewer

Does this PR:

  • Place non-StackStorm code into the lib directory?
  • Have unit tests for the action/sensor and lib layers?
  • Have clear and obvious action parameter names and descriptions?

limit_by_projects was returning a parse_meta_params error
Copy link

codecov bot commented Nov 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.24%. Comparing base (eaee134) to head (a7c4041).
Report is 15 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #287   +/-   ##
=======================================
  Coverage   98.24%   98.24%           
=======================================
  Files         239      239           
  Lines        9028     9028           
  Branches      529      529           
=======================================
  Hits         8870     8870           
  Misses        137      137           
  Partials       21       21           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@anish-mudaraddi anish-mudaraddi merged commit bca5d06 into main Nov 19, 2024
14 checks passed
@anish-mudaraddi anish-mudaraddi deleted the fix_server_datetime_search branch November 19, 2024 16:10
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