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

(Suspended) Download Attachments on click #201

Merged
merged 30 commits into from
Aug 16, 2021

Conversation

JayKim94
Copy link
Collaborator

@JayKim94 JayKim94 commented Jun 26, 2021

Before this PR can be merged, the following checkboxes have to be ticked

  • This code belongs to the backend and is tested, or tests are not useful
  • New functions with more than 5 lines have docstrings
  • New code has been integrated with the MessengerService if appropiate
  • Changes introduce no commented out code or Console.Writeline() statements used for debugging

Related issues:

closes #149

in development

@JayKim94 JayKim94 added this to the Sprint 2 milestone Jun 26, 2021
@JayKim94 JayKim94 self-assigned this Jun 26, 2021
@JayKim94 JayKim94 linked an issue Jun 26, 2021 that may be closed by this pull request
5 tasks
@JayKim94
Copy link
Collaborator Author

JayKim94 commented Jun 27, 2021

@JonasMuehlmann H e l p
image
image

Everything else is bound with Frontend (UI, Command, Model etc.), I just can't test it because of this.

@JonasMuehlmann
Copy link
Owner

JonasMuehlmann commented Jun 27, 2021

@JonasMuehlmann H e l p
image
image

Can you try logging the output of Path.GetFullPath()?

It says actually in the logging, but that outputs valid one
image
image

The path accessed is a valid one.

@JonasMuehlmann
Copy link
Owner

@JayKim94 Do the tests pass on your system?There might be some differences between Linux and Windows that I did not account for.

@JayKim94
Copy link
Collaborator Author

@JonasMuehlmann I also don't know if it worked before because at that time we didn't have any View to show them. I just assumed it would work because of the test results.

@JayKim94
Copy link
Collaborator Author

Tests also failed.

@JonasMuehlmann
Copy link
Owner

Tests also failed.

Interesting, wanna have a chat about this on teams?

@JayKim94
Copy link
Collaborator Author

Temporarily closed until we figure out how to deal with local files in UWP

Issue #149 is also on suspend

@JayKim94 JayKim94 added discussion Requires opinions of team members status: not started and removed status: in development labels Jun 27, 2021
@JayKim94 JayKim94 changed the title Download Attachments on click (Suspended) Download Attachments on click Jun 27, 2021
@JayKim94 JayKim94 changed the title (Suspended) Download Attachments on click *(Suspended)* Download Attachments on click Jun 27, 2021
@JayKim94 JayKim94 changed the title *(Suspended)* Download Attachments on click (Suspended) Download Attachments on click Jun 27, 2021
@robbelDev robbelDev self-assigned this Aug 15, 2021
@JayKim94 JayKim94 mentioned this pull request Aug 16, 2021
2 tasks
@robbelDev robbelDev merged commit fc6f9bf into unstable Aug 16, 2021
@JonasMuehlmann JonasMuehlmann deleted the download-attachment-on-click branch September 24, 2021 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Requires opinions of team members frontend status: in development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bind ChatPage for attachments in message
4 participants