Project

General

Profile

Actions

Feature #26151

closed

add graphql base classes

Added by Timo Goebel almost 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
API
Target version:
-
Difficulty:
trivial
Triaged:
No
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 1 (1 open0 closed)

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

Actions
Actions #1

Updated by Timo Goebel almost 6 years ago

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

Updated by The Foreman Bot almost 6 years ago

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

Updated by Timo Goebel almost 6 years ago

  • Status changed from Ready For Testing to Closed
Actions #4

Updated by Ondřej Pražák almost 6 years ago

  • Fixed in Releases 1.22.0 added
Actions

Also available in: Atom PDF