Skip to content

Gallery: index updateOne #385

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.

Sign up for GitHub

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

Jump to bottom
Open
mdrlzy wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Gallery: index updateOne #385

mdrlzy wants to merge 2 commits into from

Conversation

@mdrlzy
Copy link
Member

@mdrlzy mdrlzy commented Sep 21, 2023 •
edited by kirillt
Loading

In both Resource Grid and Gallery view:

  • Use update_one instead of update_all when we delete a resource or return from an external editor
  • When multiple selection is used, use update_one with each of selected resources

In Gallery view:

  • Update the current resource with new id

kirillt
kirillt reviewed Sep 21, 2023
View reviewed changes
app/build.gradle
//For local developments of arklib-android:
//def arklibVersion = '999'
def arklibVersion = '0.2.6'
def arklibVersion = '999'
Copy link
Member

@kirillt kirillt Sep 21, 2023

Choose a reason for hiding this comment

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

just to not forget

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 18, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@kirillt kirillt mentioned this pull request Sep 20, 2023
Open
@kirillt
Copy link
Member

kirillt commented Jan 3, 2024

Blocked by:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kirillt kirillt kirillt left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mdrlzy @kirillt