ampproject / ampproject/amphtml
Change parameters for rcmwidget extension for amp-ad
- 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
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