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-124277: Localize Gnav Sign In Link #432

Merged
merged 1 commit into from
Feb 2, 2023
Merged

Conversation

janaki-r-bhagwath
Copy link
Contributor

- Call decorateLinks for gnav sign in

Resolves: MWPW-124277
@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Jan 31, 2023

Hello, I'm Franklin Bot and I will run some test suites that validate the page speed.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-run PSI Checks

@janaki-r-bhagwath janaki-r-bhagwath added the trivial PR doesn't require E2E testing by a reviewer label Jan 31, 2023
@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Jan 31, 2023

Page Score PSI Audit Google
/fr/drafts/bhagwath/demo/demo?martech=off SI FCP LCP TBT CLS PSI
/ SI FCP LCP TBT CLS PSI
/fragments/cmillar/my-modal SI FCP LCP TBT CLS PSI

@codecov
Copy link

codecov bot commented Jan 31, 2023

Codecov Report

Merging #432 (f8c01cd) into main (3bac1a3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #432   +/-   ##
=======================================
  Coverage   98.90%   98.90%           
=======================================
  Files          80       80           
  Lines       21016    21017    +1     
=======================================
+ Hits        20786    20787    +1     
  Misses        230      230           
Impacted Files Coverage Δ
libs/blocks/gnav/gnav.js 97.65% <100.00%> (+<0.01%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@janaki-r-bhagwath janaki-r-bhagwath merged commit e282c6d into main Feb 2, 2023
@janaki-r-bhagwath janaki-r-bhagwath deleted the link-transform branch February 2, 2023 20:54
sukamat pushed a commit to sukamat/milo that referenced this pull request Feb 8, 2023
- Call decorateLinks for gnav sign in

Resolves: MWPW-124277
sukamat pushed a commit to sukamat/milo that referenced this pull request Feb 8, 2023
base project page

placeholder fg css

connect to SP and init floodgate project

Reads the floodgate project and injects SP data

Add asset support to library

Add Franklin customer library support

Remove 3rd party libraries if not used

Add utils test coverage (adobecom#427)

Co-authored-by: Ryan Clayton <rclayton@adobe.com>

A simple `reading-time` block to estimate the reading time of the article (adobecom#356)

* A simple  block to estimate the reading time of the article

* reading-time: typo in the css

* reading-time: small change to the css

* reading-time PR review changes

* added placeholders for localization of the '5 min read' text

* [reading-time]: add a placeholder key for singular reading time value

* [reading-time]: less strict scoping for css

* [reading-time]: less strict scoping for css

* [reading-time]: text-align center only if it's in a center section

* reading-time tests added

* reading-time tests added

Fixing bug for faas import from link clicked. (adobecom#424)

* Fixing bug for faas import from link clicked.

* adding ||= sytex.

* same as before.

MWPW-122611 - [GH Action] Old browser function detection (adobecom#423)

* MWPW-122611 - [GH Action] Old browser function detection

- add a Github action to check for unsupported functions

* MWPW-122611 - [GH Action] Old browser function detection

- add a Github action to check for unsupported functions

* MWPW-122611 - [GH Action] Old browser function detection

- add a Github action to check for unsupported functions

* MWPW-122611 - [GH Action] Old browser function detection

- add a Github action to check for unsupported functions

* MWPW-122611 - [GH Action] Old browser function detection

- add a Github action to check for unsupported functions

* MWPW-122611 - [GH Action] Old browser function detection

- add a Github action to check for unsupported functions

* MWPW-122611 - [GH Action] Old browser function detection

- add a Github action to check for unsupported functions

* MWPW-122611 - [GH Action] Old browser function detection

- add a Github action to check for unsupported functions

---------

Co-authored-by: Aaron Mauchley <mauchley@adobe.com>

removing display none from accordion (adobecom#433)

Media & Z-Pattern updates - content margins and 2-5up grid support (adobecom#429)

* updates to media to support section meta two-five ups

* updated default margins

* Added detail styles to text-transform:uppercase
Removed redundant asside styles.

* test coverate on image w/ out alt text

---------

Co-authored-by: thi64146 <thi64146@adobe.com>
Co-authored-by: Ryan Clayton <rgclayton@gmail.com>

Update node to v18 (adobecom#440)

MWPW-125458 - Adding chat icon (adobecom#439)

Adding chat icon for bacom use

Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>

MWPW-124813 Prevent long breadcrumbs overlapping search (adobecom#434)

MWPW-124277: Localize Gnav Sign In Link (adobecom#432)

- Call decorateLinks for gnav sign in

Resolves: MWPW-124277

add ignores similar to loc

project details table markup updated

removed console logs

updated admin page to keep just the copy action

removed copy files placeholder code

clean up and loc config import fix

cleanup

rename app.js to ui.js

floodgate content display updated

cleanup

removed window.projectDetail

refactor
chrischrischris pushed a commit to adobecom/milo-target that referenced this pull request May 22, 2023
- Call decorateLinks for gnav sign in

Resolves: MWPW-124277
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
trivial PR doesn't require E2E testing by a reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants