Skip to content
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

Improve structure and tests #2

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

Skullbock
Copy link
Contributor

Questa PR estende e sostituisce #1 di @KistianLentino99

Idee di base

  • Possibilità di passare ulteriori configurazioni (integrative o sostitutive) al metodo send, come secondo parametro
  • Tutti i parametri "extra" per l'invio, come simplified di acube, vanno passati o al costrutture del sender, o più probabilmente come secondo parametro del metodo send
  • Il metodo send ora ritorna un'oggetto Result, che è un semplice array associativo "esteso" con i dettagli che ogni sender può o non può dover ritornare. Mi sembrava il modo più semplice per gestire la cosa.
  • Ho aggiunto un'interfaccia ProvidesConfigurationKeys che determina se l'adapter ritorna le stringhe di configurazioni possibili per il sender stesso.

@KristianLentino99
Copy link

@Skullbock il co-founder di Acube mi suggerisce di gestire anche lo status code della risposta

@Skullbock
Copy link
Contributor Author

Ottima idea.
Per caso riesci a farti dare un set di codici per testing, visto che stiamo scrivendo la libreria? 😅🤞🏻

@KristianLentino99
Copy link

@Skullbock https://api-sandbox.acubeapi.com/docs.html qui dovrebbe esserci tutta la struttura delle API .

@Skullbock
Copy link
Contributor Author

Si, intendevo se potevano fornirci le credenziali per un ambiente sand box senza dover acquistare il servizio

@KristianLentino99
Copy link

@Skullbock se vuoi posso girarti in privato le mie che ho usato esclusivamente per questo progetto, altrimenti scrivo una mail per farcele dare condivise

@Skullbock
Copy link
Contributor Author

Preferite la seconda così è più "pulito" e anche ufficioso 👌🏻

@KristianLentino99
Copy link

@Skullbock Ok , chiedo! 💪🏻💪🏻

@Skullbock
Copy link
Contributor Author

Grazie 🤞🏻🙏🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants