Skip to content

Answer:59#1271

Closed
testaccount00075433 wants to merge 1 commit intotomalaforge:mainfrom
testaccount00075433:Answer59
Closed

Answer:59#1271
testaccount00075433 wants to merge 1 commit intotomalaforge:mainfrom
testaccount00075433:Answer59

Conversation

@testaccount00075433
Copy link
Copy Markdown

Checklist for challenge submission

  • Start your PR message with Answer:${challenge_number}

Warning:

  • If you want feedback or review, you must support the project on GitHub:

Alternatively, you can still submit your PR to join the list of answered challenges or to be reviewed by a community member. 🔥

@github-actions github-actions Bot added 59 defer answer answer sponsor sponsor the project to be reviewed PR requests a review labels Apr 3, 2025
@testaccount00075433 testaccount00075433 mentioned this pull request Apr 3, 2025
loader: async ({ request }) =>
request.isCardOpened
? await fetch(this.URL).then((res) => res.json())
: [],
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

i think how angular designed the API is to return undefined when you don't want to trigger the API call but nice solution.

@tomalaforge tomalaforge removed the to be reviewed PR requests a review label Apr 4, 2025
@testaccount00075433 testaccount00075433 closed this by deleting the head repository Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

59 defer answer answer sponsor sponsor the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants