ampproject / ampproject/amphtml

Change parameters for rcmwidget extension for amp-ad

Open
#39,158 3 comments 0 reactions 0 assignees View on GitHub
INTENT TO IMPLEMENT Stale
Dominant language
JavaScript
Stars
14.9k
Forks
4.1k
PR merge metrics
No merged PRs in 30d

Description

### Summary

Hi!
One of parameters (`projectId`) of out widget is outdated and should be removed from the list. And I want to contribute this change into amp-ad.

~And 1 param should be added to required params: `layoutType`.~ We don't need it anymore.

Widget is used to place dynamic blocks with the most relevant materials on Rambler sites.

### Design Document

_No response_

### Motivation

One of params became unnecessary, so I want to remove it from the list of params.

### Alternative Solutions

no

### Launch Tracker

_No response_

### Notifications

/cc @ampproject/wg-approvers

Contributor guide

Open the contributing guide

Research direction

The issue identifies the rcmwidget configuration for the amp-ad extension but names no file or test. Start by locating the rcmwidget parameter definition and confirm how projectId is listed; verify that layoutType remains excluded. Done means the parameter list no longer includes projectId and the relevant checks still pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend, web-dev
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.