Open-event-server: Add Foreign keys to Invoice model

Created on 16 Jul 2019  路  5Comments  路  Source: fossasia/open-event-server

Is your feature request related to a problem? Please describe.
Currently, Event Invoice does not link to an order. This should be linked as it would help in the integration of invoice payments with the order workflow

Additional context
Working on it.

feature

Most helpful comment

Issue-Label Bot is automatically applying the label feature to this issue, with a confidence of 0.93. Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

All 5 comments

Issue-Label Bot is automatically applying the label feature to this issue, with a confidence of 0.93. Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

Now that I think of it, this will require change in every query where Order is used and in relationship as well. This will be a no go. Please create a new table for it and add order as foreign key, which will both save the work and not break existing code

Reverting the PR

@iamareebjamal Yes, I was dubious about this because of the order refactor.
Event Invoice table is already present so I'll just use it. I'll add the order foreign key.

@iamareebjamal Just a question. The orders would still have to be recorded onto the Order table even with the foreign key, right?
Wouldn't it interfere in every query where Order is used(as this order is payment for event invoice)? Please correct me if I'm wrong.

Why will order be inserted? It'll already be there in the table, right?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

iamareebjamal picture iamareebjamal  路  4Comments

dr0pdb picture dr0pdb  路  4Comments

schedutron picture schedutron  路  4Comments

aditya1702 picture aditya1702  路  4Comments

aviaryan picture aviaryan  路  3Comments