Skip to content

feat: Show message is metadata scanning is present #4879

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
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: Show message is metadata scanning is present #4879

wants to merge 1 commit into from

Conversation

steverydz
Copy link
Contributor

@steverydz steverydz commented Oct 11, 2024 •
edited
Loading

Done

Add a message on the listing page when updates are saved

How to QA

  • Go to https://staging-api.snapcraft.io/<SNAP_NAME>/listing
  • Make a change and save it
  • Check that there is a "Metadata updates will be processed and applied shortly" message

Testing

  • This PR has tests
  • No testing required (explain why):

Issue / Card

Fixes https://warthogs.atlassian.net/browse/WD-15740

@webteam-app
Copy link

Demo

Jenkins

demos.haus

@steverydz steverydz closed this Oct 14, 2024
@steverydz steverydz force-pushed the WD-15740-update-listing-page-message branch from d5bcf10 to a5dda8a Compare October 14, 2024 11:48
@steverydz steverydz reopened this Oct 14, 2024
@steverydz steverydz force-pushed the WD-15740-update-listing-page-message branch 3 times, most recently from 78b2e2d to 2488b94 Compare October 15, 2024 13:06
@steverydz steverydz marked this pull request as ready for review October 15, 2024 13:55
ilayda-cp
ilayda-cp approved these changes Oct 15, 2024
View reviewed changes
Copy link
Contributor

@ilayda-cp ilayda-cp left a comment

Choose a reason for hiding this comment

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

Looks good to me

@steverydz steverydz force-pushed the WD-15740-update-listing-page-message branch from 2488b94 to d3a318a Compare January 6, 2025 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ilayda-cp ilayda-cp ilayda-cp approved these changes

Assignees

No one assigned

Labels

Don't merge Review: Code +1 Review: QA +1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@steverydz @webteam-app @ilayda-cp