papercups-io/papercups

Allow to easily integrate in an existing Elixir app

Open

#210 ouverte le 4 sept. 2020

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Elixir (6 032 stars) (566 forks)batch import
help wantedneeds more info

Description

It would be great if most of this can be moved to a mix dependency where it can be easily integrated inside of another app. This repo can just use that dependency.

I also saw that the table names are not namespaced. Actually namespacing these would also help to have this coexist.

Guide contributeur

Stack technique
elixir
Domaine
backenddeveloper experience
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
ElixirMixEctoDatabase migrations
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Investigate the current project structure to understand how to extract core functionality into a separate Mix dependency. Examine mix.exs and existing database migrations for table naming conventions. Review the 3 comments on the issue for any maintainer guidance or prior discussion. Consider renaming tables with a prefix to avoid conflicts when integrating into other apps.