Summary

Move request_method_decorator into rbtools.api.resource.base

Review Request #14246 — Created Nov. 15, 2024 and updated Dec. 12, 2024, 12:01 p.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

This change moves the request_method_decorator into our base resource
definitions, since it's tightly tied to resources. It also renames it to
request_method for brevity. The old name will still work with a
deprecation warning.

Testing Done

Ran unit tests.

Commits

Summary	ID
Move request_method_decorator into rbtools.api.resource.base This change moves the `request_method_decorator` into our base resource definitions, since it's tightly tied to resources. It also renames it to `request_method` for brevity. The old name will still work with a deprecation warning. Testing Done: Ran unit tests.	a26da6b37ed6704692d984566d509dbdaf076dd8

Issues

Description	From	Last Updated
Should we add a version changed that says we moved and renamed the function?	maubin	Dec. 12, 2024, 12:01 p.m.

flake8 passed.

JSHint passed.

rbtools/api/resource/base.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Should we add a version changed that says we moved and renamed the function?
```

Commits:

	Summary	ID
	Move request_method_decorator into rbtools.api.resource.base This change moves the `request_method_decorator` into our base resource definitions, since it's tightly tied to resources. It also renames it to `request_method` for brevity. The old name will still work with a deprecation warning. Testing Done: Ran unit tests.	05de590e955924bc290b463a6d038bde03ebdd86
	Move request_method_decorator into rbtools.api.resource.base This change moves the `request_method_decorator` into our base resource definitions, since it's tightly tied to resources. It also renames it to `request_method` for brevity. The old name will still work with a deprecation warning. Testing Done: Ran unit tests.	a26da6b37ed6704692d984566d509dbdaf076dd8

Diff:

Revision 2 (+182 -104)

Show changes

	rbtools/api/decorators.py
	rbtools/api/resource/base.py
	rbtools/api/resource/diff.py
	rbtools/api/resource/diff_file_attachment.py
	rbtools/api/resource/draft_diff_commit.py
	rbtools/api/resource/file_attachment.py
	rbtools/api/resource/file_diff.py
	rbtools/api/resource/mixins.py
	5 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.