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

Add attachment with one API call #1581

Open
opqdonut opened this issue Sep 4, 2019 · 1 comment
Open

Add attachment with one API call #1581

opqdonut opened this issue Sep 4, 2019 · 1 comment
Labels
Technical Debt Improving internal product quality

Comments

@opqdonut
Copy link
Contributor

opqdonut commented Sep 4, 2019

Currently, an attachment is added and then a draft-saved command is issued to store the reference to the attachment in the application

Instead, we should have an API endpoint like /application/add-attachment?application-id=123&field-id=7 that would add the attachment to the db and emit a draft-saved event. This would simplify the logic on the UI side.

@opqdonut opqdonut added the Technical Debt Improving internal product quality label Sep 4, 2019
@Macroz
Copy link
Collaborator

Macroz commented Sep 4, 2019

Adding and removing of the attachment should be the same. There should also be no need to press save button separately.

@Macroz Macroz added this to the Internal cleanup milestone Mar 20, 2022
@meericsc meericsc moved this to Ideas in REMS Task Board May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Technical Debt Improving internal product quality
Projects
Status: Ideas
Development

No branches or pull requests

2 participants