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

Task/leip 164 upload images #289

Merged
merged 36 commits into from
Nov 15, 2023
Merged

Task/leip 164 upload images #289

merged 36 commits into from
Nov 15, 2023

Conversation

hb0
Copy link
Contributor

@hb0 hb0 commented Oct 23, 2023

Recommendation:
Use commit-wise reviewing or else you will be confused because Github's "Files changed" view will think I changed a previously named class "FileDao" but I've renamed the old class to a more correct name "FileIOHandler" and added a new FileDao class which follows the recommended Android architecture for Room and data access.

@hb0 hb0 self-assigned this Oct 23, 2023
@hb0 hb0 force-pushed the task/LEIP-164_upload-images branch 2 times, most recently from 9d7c5b5 to 74837a7 Compare October 23, 2023 16:28
@hb0 hb0 force-pushed the task/LEIP-164_upload-images branch from 223f2b2 to 827cd8b Compare October 24, 2023 14:38
@hb0 hb0 force-pushed the task/LEIP-164_upload-images branch from 26ba803 to 11e08f2 Compare October 25, 2023 09:22
@hb0 hb0 force-pushed the task/LEIP-164_upload-images branch from 11e08f2 to d6ee15d Compare October 25, 2023 09:24
@hb0 hb0 force-pushed the task/LEIP-164_upload-images branch from 3467764 to 79a1f05 Compare October 26, 2023 12:07
@hb0 hb0 force-pushed the task/LEIP-164_upload-images branch from e54d3d5 to 48c7123 Compare October 26, 2023 13:27
@hb0 hb0 force-pushed the task/LEIP-164_upload-images branch from 5604a32 to 2c120f8 Compare October 31, 2023 15:04
@hb0 hb0 marked this pull request as ready for review October 31, 2023 15:31
Copy link
Contributor Author

@hb0 hb0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pre-reviewed

build.gradle Outdated Show resolved Hide resolved
build.gradle Outdated Show resolved Hide resolved
persistence/build.gradle Show resolved Hide resolved
@hb0 hb0 force-pushed the task/LEIP-164_upload-images branch from d107a0a to 241f558 Compare October 31, 2023 15:51
Copy link
Member

@muthenberg muthenberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small changes.

@hb0 hb0 force-pushed the task/LEIP-164_upload-images branch from 2d347bb to c726a86 Compare November 13, 2023 15:51
Copy link
Contributor Author

@hb0 hb0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re-reviewed

@hb0 hb0 merged commit 2f286f6 into main Nov 15, 2023
1 of 2 checks passed
@hb0 hb0 deleted the task/LEIP-164_upload-images branch November 15, 2023 09:11
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