Skip to content
Snippets Groups Projects

Add support for api/parameters endpoint

Merged Add support for api/parameters endpoint
SETRA-70-params-api into master
1 unresolved thread
Merged Andreas Ellewsen requested to merge SETRA-70-params-api into master
1 unresolved thread

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Trond Aasan approved this merge request

    approved this merge request

  • Andreas Ellewsen added 1 commit

    added 1 commit

    • 4762304b - Add support for api/parameters endpoint

    Compare with previous version

  • Andreas Ellewsen resolved all threads

    resolved all threads

  • Ruben.Havre
    Ruben.Havre @Ruben.Havre started a thread on the diff
    • setra_client/client.py
    262 275 response = self.get(url)
    263 276 return self.object_or_data(BatchErrors, response.json())
    264 277
    278 def get_parameters(self, interface: str = None):
    279 """Make a GET request to the parameters endpoint"""
    280 url = self.urls.parameters()
    281 params = None
    282 if interface:
    • Ruben.Havre
      Ruben.Havre @Ruben.Havre ·
      Contributor

      Er params ikke påkrevd? Hva skjer dersom den utelates, eller man bruker en annen interface enn ens egen?

    • Andreas Ellewsen
      Andreas Ellewsen @ae ·
      Author Contributor

      params er litt misvisende her. Jeg endrer den til queryparams så det er klarere. De brukes til å filtrere og man kan kun filtrere på interface for øyeblikket.

      Hvis man spør om en annen interface enn sin egen får man svar med alt det man ikke har rett til å vite, men jeg er ikke veldig bekymret siden vi har kontroll på hele flyten. Vi burde nok endre på tilgangsstyringen i setra apiet, men det vil føre til at vi må ha planer i gravitee for forskjellige interfacer, og vi bør nok sette oss ned og planlegge litt hvordan det gjøres best før vi setter i gang med det.

    • Ruben.Havre
      Ruben.Havre @Ruben.Havre ·
      Contributor

      Tror jeg skrev feil, mente interface parameteret, ikke params. :)

    • Andreas Ellewsen
      Andreas Ellewsen @ae ·
      Author Contributor

      Jeg lot det være mulig å få ut alle hvis man ikke spesifiserte interface. Ser ikke nytteverdien akkurat nå, men endepunkte støtter det så det ble litt rart å legge på en falsk begrensning i klienten.

    • Please register or sign in to reply
  • Ruben.Havre approved this merge request

    approved this merge request

  • Andreas Ellewsen added 1 commit

    added 1 commit

    • 2279758a - Add support for api/parameters endpoint

    Compare with previous version

  • Trond Aasan
  • Andreas Ellewsen added 1 commit

    added 1 commit

    • 230b6658 - Simplify data to object conversion

    Compare with previous version

  • Andreas Ellewsen merged

    merged

  • Andreas Ellewsen mentioned in commit 715a429c

    mentioned in commit 715a429c

    • Please register or sign in to reply