avo icon indicating copy to clipboard operation
avo copied to clipboard

feature: resource adapters

Open Paul-Bob opened this issue 1 year ago • 2 comments

Description

Fixes #1418

Breaking changes

  • Avo::BaseResource -> Avo::Resources::Base
  • Avo::Resources::ActiveRecord will replace the old Avo::BaseResource, now Avo::Resources::Base will keep what's shared by all resources types.
  • Avo::ResourcesController -> Avo::ActiveRecordResourcesController and there is also a new controller type: Avo::HttpResourcesController

Todo

  • [ ] HTTP resource generator
  • [ ] Fix Http resources menu duplication

Checklist:

  • [ ] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] I have added tests that prove my fix is effective or that my feature works

Screenshots & recording

Manual review steps

  1. Step 1
  2. Step 2

Manual reviewer: please leave a comment with output from the test if that's the case.

Paul-Bob avatar Feb 06 '24 10:02 Paul-Bob

Code Climate has analyzed commit 64a7ee6f and detected 11 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 11

View more on Code Climate.

codeclimate[bot] avatar Feb 06 '24 10:02 codeclimate[bot]

This PR has been marked as stale because there was no activity for the past 15 days.

github-actions[bot] avatar Feb 29 '24 01:02 github-actions[bot]

Closing for now,

adrianthedev avatar Apr 24 '24 22:04 adrianthedev