Skip to content

Conversation

@varunguleriaCodes
Copy link

Motivation

Dependencies update + Dark Mode
Description
Update the dependencies to the latest version + adds a dark mode into the page, the dark mode looks like this->
{C1AC2E01-4E66-49B7-9DAE-44694A04C90E}

Closes #issue_number
#407
Steps to test or reproduce

@varunguleriaCodes varunguleriaCodes requested a review from a team as a code owner December 2, 2024 05:54
@CLAassistant
Copy link

CLAassistant commented Dec 2, 2024

CLA assistant check
All committers have signed the CLA.

const keyFromCaseMap = casingMap[fieldName];
if (keyFromCaseMap === undefined) {
throw Error(`casingMap[${fieldName}] not defined`);
throw Error(`casingMap[${String(fieldName)}] not defined`);
Copy link
Member

Choose a reason for hiding this comment

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

why is this required? fieldName should be type string

Copy link
Author

Choose a reason for hiding this comment

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

Hi @nflaig when I ran the build command this error was showing up "Implicit conversion of a 'symbol' to a 'string' will fail at runtime. Consider wrapping this expression in 'String(...)'. " to overcome this I applied typecasting

Copy link
Member

@nflaig nflaig left a comment

Choose a reason for hiding this comment

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

merge conflicts need to be resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants