Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Investigate adding All type to RemoveTimeFrame #26179

Closed
mavduevskiy opened this issue Jul 25, 2022 · 2 comments
Closed

Investigate adding All type to RemoveTimeFrame #26179

mavduevskiy opened this issue Jul 25, 2022 · 2 comments
Assignees
@mavduevskiy
Labels
needs:triage Issue needs triage

Comments

@mavduevskiy
Copy link
Contributor

mavduevskiy commented Jul 25, 2022 •
edited by data-sync-user
Loading

Relates to this discussion.

I have concerns of not using the All type LongRange parameters, since the provider doesn't need any parameters to remove all items. But adding All type would make the code more readable.

┆Issue is synchronized with this Jira Task
@mavduevskiy mavduevskiy self-assigned this Jul 25, 2022
@github-actions github-actions bot added the needs:triage Issue needs triage label Jul 25, 2022
@mavduevskiy mavduevskiy mentioned this issue Jul 25, 2022
Merged
4 tasks
@stale
Copy link

stale bot commented Jan 21, 2023

See: #17373 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Jan 21, 2023
@boek
Copy link
Contributor

boek commented Jan 30, 2023

Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1813801

Change performed by the Move to Bugzilla add-on.

@boek boek closed this as completed Jan 30, 2023
@stale stale bot removed the wontfix label Jan 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mavduevskiy mavduevskiy

Labels
needs:triage Issue needs triage
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@boek @mavduevskiy