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

MWPW-146263 - [Bulk Publish v2] Job Info UI #2158

Merged
merged 15 commits into from
May 1, 2024

Conversation

Sartxi
Copy link
Contributor

@Sartxi Sartxi commented Apr 17, 2024

Display job info such as progress, status, and give option to copy a jobs invocation ID in the Job Results Panel. Also adds error filtering and error rework functionality.

Resolves: MWPW-146263

Test URLs:

@Sartxi Sartxi added new-feature New block or other feature needs-verification PR requires E2E testing by a reviewer zero-impact labels Apr 17, 2024
Copy link
Contributor

aem-code-sync bot commented Apr 17, 2024

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

Copy link
Contributor

aem-code-sync bot commented Apr 17, 2024

Page Scores Audits Google
/tools/bulk-publish-v2?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync aem-code-sync bot temporarily deployed to sarchibeque/mwpw-146263-job April 23, 2024 21:02 Inactive
@rgclayton
Copy link
Contributor

Looking good Sean!

If doing multiple jobs, having the most recent job be at the top of the list. Otherwise I have to scroll a bunch to get to my most recent job links. Or maybe each job batch can be collapsible/expandable?

image

Copy link
Contributor

This pull request is not passing all required checks. Please see this discussion for information on how to get all checks passing. Inconsistent checks can be manually retried. If a test absolutely can not pass for a good reason, please add a comment with an explanation to the PR.

Copy link

codecov bot commented Apr 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.67%. Comparing base (3b3149a) to head (42c09c6).
Report is 5 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage    #2158      +/-   ##
==========================================
- Coverage   96.15%   95.67%   -0.49%     
==========================================
  Files         165      165              
  Lines       43591    43534      -57     
==========================================
- Hits        41916    41651     -265     
- Misses       1675     1883     +208     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aem-code-sync aem-code-sync bot temporarily deployed to sarchibeque/mwpw-146263-job April 24, 2024 20:19 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to sarchibeque/mwpw-146263-job April 25, 2024 15:05 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to sarchibeque/mwpw-146263-job April 26, 2024 18:06 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to sarchibeque/mwpw-146263-job April 29, 2024 18:57 Inactive
@elan-tbx elan-tbx added verified PR has been E2E tested by a reviewer and removed needs-verification PR requires E2E testing by a reviewer labels Apr 30, 2024
@Blainegunn Blainegunn merged commit c65ad98 into stage May 1, 2024
13 of 14 checks passed
@Blainegunn Blainegunn deleted the sarchibeque/mwpw-146263-job branch May 1, 2024 22:12
Blainegunn added a commit that referenced this pull request May 2, 2024
* MWPW-147027: CTA flowing out of mini compare card (due to long strings) (#2217)

* MWPW-147027: CTA flowing out of mini compare card (due to long strings)

* footer fix

* Update merch-card.js

* Update merch-card.js

* MWPW-147149: Prevent multiple download/upgrade CTAs (#2226)

* MWPW-147149: Prevent multiple download links

in the same marquee or merch-card

* add upgrade use case.

* add comment

* typo fix

* improve href attribute setting

* commerce.js from develop

* MWPW-146845 Setup Footer Promo for Localization (#2236)

* MWPW-146263 - [Bulk Publish v2] Job Info UI (#2158)

* add job info to job processes

* add job info updates

* ui dialing

* finish up UI and refactored a little

* update label

* unit tests to support new functionality

* package index jobs into one

* feedback from demo

* fix unit test

* code cov

* get full code cov

* can test copy to clipboard in process component

* fix tabs test

* Adding additional IMS Scope for A.com Client IDs  (#2249)

* MWPW-146784 Add new default IMS scopes

* MWPW-146784 Add new default IMS scopes

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>

* MWPW-146070: Upgrade Universal Nav to Version 1.1 in Milo (#2250)

* MWPW-146070: Upgrade unav to 1.1

* MWPW-146784 Add new default IMS scopes

* MWPW-146784 Add new default IMS scopes

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>

---------

Co-authored-by: Axel Cureno Basurto <axelcureno@gmail.com>
Co-authored-by: ilyas Stéphane Türkben <tuerkben@adobe.com>
Co-authored-by: Megan Thomas <methomas@adobe.com>
Co-authored-by: Sean Archibeque <sean.archibeque@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: nishantka <126539566+nishantka@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature New block or other feature Ready for Stage verified PR has been E2E tested by a reviewer zero-impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants