Skip to content

Add aria_custom_resource and related models #16

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

Merged
merged 32 commits into from
Jul 9, 2024
Merged

Conversation

davidfischer-ch
Copy link
Owner

@davidfischer-ch davidfischer-ch commented Jun 28, 2024

Fix #8

  • Custom Resource Core
  • Properties
    • Core property attributes
    • Implement Default to/from API
    • Test Default
    • Integrate Properties to/from API inside Custom Resource to/from API
    • Test with some content (in custom resource acceptance test -> especially nil value (undefined) and other subtle edge cases)
    • Properties Ordering
    • Implement OneOf to/from API
    • Test OneOf
  • CRUD Actions
    • Create Action
    • Read Action
    • Update Action
    • Delete Action
    • Make Update Action optional?
  • Additional Actions 🚧
  • Custom Form inside (tm) ...
  • Update example in doc

Fix #18

  • Resource Action

@Towl
Copy link

Towl commented Jul 4, 2024

Hello,

Here is an example that could be useful: https://stackoverflow.com/questions/25182923/serialize-a-map-using-a-specific-order

@Towl
Copy link

Towl commented Jul 4, 2024

And this : https://pkg.go.dev/gitlab.com/c0b/go-ordered-json

@davidfischer-ch
Copy link
Owner Author

I finally made it! Both unit and acceptances tests are showing that ordering of properties is preserved!

@davidfischer-ch davidfischer-ch added enhancement New feature or request go Pull requests that update Go code labels Jul 5, 2024
@davidfischer-ch davidfischer-ch merged commit f9878a4 into main Jul 9, 2024
12 checks passed
@davidfischer-ch davidfischer-ch deleted the custom-resources branch July 10, 2024 14:03
@davidfischer-ch davidfischer-ch self-assigned this Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add aria_resource_action managed resource Add aria_custom_resource managed resource
2 participants