Project

General

Profile

Feature #26151

add graphql base classes

Added by Timo Goebel 6 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
API
Target version:
-
Difficulty:
trivial
Triaged:
No
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

It's recommended to create our own base classes instead of inheriting from graphql library classes directly. (similar to application record vs inheriting from ActiveRecord::Base)
We should introduce this as early as possible so plugins don't require changes later.


Related issues

Related to Foreman - Tracker #22109: Foreman should support a graphql apiNew

Associated revisions

Revision 6cf766e6 (diff)
Added by Timo Goebel 6 months ago

fixes #26151 - graphql: add base classes

History

#1 Updated by Timo Goebel 6 months ago

  • Related to Tracker #22109: Foreman should support a graphql api added

#2 Updated by The Foreman Bot 6 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6508 added

#3 Updated by Timo Goebel 6 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by Ondřej Pražák 6 months ago

  • Fixed in Releases 1.22.0 added

Also available in: Atom PDF