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

Allow to download files linked to a bank operation #73

Merged
merged 3 commits into from
Apr 6, 2015

Conversation

frankrousseau
Copy link

I open the #71 PR again but this time on the right branch. Here is the description of what I did:

  • When a file is linked to the bank operation, the amount turns into a link that allows to download that file.
  • I removed the old icon to change it by the "Mes comptes" one. Because:
    • We will soon remove the "Mes comptes application" because we want to remove Weboob from the Data Indexer.
    • The Mes comptes SVG icon is less buggy than the Kresus SVG icon.

Frank Rousseau added 3 commits April 6, 2015 22:59
When displaying operations, if there is a file linked to an operation (a bill
most of the time), it adds a link tag on the operation amount.
This icon works better in the marketplace
The field is set on the Bank Operation model.
@bnjbvr
Copy link
Collaborator

bnjbvr commented Apr 6, 2015

Wonderful! Thank you, I will try it with my Bouygues Telecom bills, as I see Konnectors can make the linking.

bnjbvr added a commit that referenced this pull request Apr 6, 2015
Allow to download files linked to a bank operation
@bnjbvr bnjbvr merged commit 18c74d7 into kresusapp:incoming Apr 6, 2015
@frankrousseau
Copy link
Author

You have to take the version of my repository. But it didn't work yet (only a few bills are linked). I have to write tests to make it work properly.

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