Skip to content

feat: add nodules support for interactive challenges #1674

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.

Sign up for GitHub

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: add nodules support for interactive challenges #1674

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1e54f05
feat: add nodules support for interactive challenges and update pubsp…
Sembauke Oct 17, 2025
7b995f1
fix: make nodule values nullable
Sembauke Oct 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
97 changes: 97 additions & 0 deletions mobile-app/lib/models/learn/challenge_model.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,9 @@ class Challenge {
final EnglishAudio? audio;
final Scene? scene;

// Nodules for interactive challenges
final List<Nodules>? nodules;

final List<Question>? questions;

// Challenge Type 15 - Odin
Expand All @@ -92,6 +95,7 @@ class Challenge {
required this.helpCategory,
this.explanation,
required this.transcript,
this.nodules,
this.questions,
this.assignments,
this.fillInTheBlank,
Expand Down Expand Up @@ -127,6 +131,9 @@ class Challenge {
files: (data['challengeFiles'] ?? [])
.map<ChallengeFile>((file) => ChallengeFile.fromJson(file))
.toList(),
nodules: (data['nodules'] ?? [])
.map<Nodules>((nodule) => Nodules.fromJson(nodule))
.toList(),
questions: (data['questions'] as List).isNotEmpty
? (data['questions'] as List)
.map<Question>((q) => Question.fromJson(q))
Expand Down Expand Up @@ -225,6 +232,96 @@ class Hooks {
}
}

sealed class StringOrList {}

enum NoduleType {
paragraph('paragraph'),
interactiveEditor('interactiveEditor');

final String type;

const NoduleType(this.type);
}

class NoduleTypeString extends StringOrList {
final String value;

NoduleTypeString(this.value);
}

class NoduleTypeList extends StringOrList {
final List<InterActiveFile> value;

NoduleTypeList(this.value);
}

class Nodules {
final NoduleType type;
final StringOrList data;

Nodules({
required this.type,
required this.data,
});

String asString() {
return (data as NoduleTypeString).value;
}

List<InterActiveFile> asList() {
return (data as NoduleTypeList).value;
}

factory Nodules.fromJson(Map<String, dynamic> data) {
final noduleType = NoduleType.values.firstWhere(
(type) => type.type == data['type'],
orElse: () => throw ArgumentError('Invalid nodule type: ${data['type']}'),
);

final noduleData = data['data'];
if (noduleData is String) {
return Nodules(
type: noduleType,
data: NoduleTypeString(noduleData),
);
} else if (noduleData is List) {
return Nodules(
type: noduleType,
data: NoduleTypeList(
noduleData
.map<InterActiveFile>(
(item) => InterActiveFile.fromJson(item),
)
.toList(),
),
);
} else {
throw ArgumentError(
'Invalid nodule data type: ${noduleData.runtimeType}');
}
}
}

class InterActiveFile {
final String name;
final String contents;
final String ext;

InterActiveFile({
required this.ext,
required this.name,
required this.contents,
});

factory InterActiveFile.fromJson(Map<String, dynamic> data) {
return InterActiveFile(
ext: data['ext'],
name: data['name'],
contents: data['contents'],
);
}
}

class Question {
final String text;
final List<Answer> answers;
Expand Down
30 changes: 30 additions & 0 deletions mobile-app/lib/ui/views/learn/challenge/templates/multiple_choice/multiple_choice_view.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import 'package:freecodecamp/ui/views/learn/widgets/quiz_widget.dart';
import 'package:freecodecamp/ui/views/learn/widgets/transcript_widget.dart';
import 'package:freecodecamp/ui/views/learn/widgets/youtube_player_widget.dart';
import 'package:freecodecamp/ui/views/news/html_handler/html_handler.dart';
import 'package:phone_ide/phone_ide.dart';
import 'package:stacked/stacked.dart';

class MultipleChoiceView extends StatelessWidget {
Expand Down Expand Up @@ -59,6 +60,35 @@ class MultipleChoiceView extends StatelessWidget {
),
),
),
if (challenge.nodules!.isNotEmpty)
ChallengeCard(
title: 'Lesson',
child: Column(
children: [
...challenge.nodules!.map(
Copy link

Copilot AI Oct 18, 2025

Choose a reason for hiding this comment

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

challenge.nodules is declared nullable on the model; using the non-null assertion (!) here risks a runtime exception if it is null. Use a null-safe check and mapping (e.g., if ((challenge.nodules?.isNotEmpty ?? false)) and ...(challenge.nodules ?? const []).map(...)), or make the model field non-nullable with a default empty list.

Suggested change
if (challenge.nodules!.isNotEmpty)
ChallengeCard(
title: 'Lesson',
child: Column(
children: [
...challenge.nodules!.map(
if ((challenge.nodules?.isNotEmpty ?? false))
ChallengeCard(
title: 'Lesson',
child: Column(
children: [
...(challenge.nodules ?? const <Nodule>[]).map(

Copilot uses AI. Check for mistakes.
(nodule) {
if (nodule.type == NoduleType.paragraph) {
return Column(
children: parser.parse(nodule.asString()),
);
} else if (nodule.type ==
NoduleType.interactiveEditor) {
return Editor(
options: EditorOptions(
takeFullHeight: false,
showLinebar: false,
),
defaultLanguage: nodule.asList()[0].ext,
defaultValue: nodule.asList()[0].contents,
path: '/');
} else {
return const SizedBox.shrink();
}
},
)
],
),
),
if (challenge.videoId != null) ...[
ChallengeCard(
title: 'Watch the Video',
Expand Down
6 changes: 3 additions & 3 deletions mobile-app/lib/ui/views/learn/landing/landing_viewmodel.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -243,9 +243,9 @@ class LearnLandingViewModel extends BaseViewModel {

List superBlocks = res.data['superblocks'];
for (int i = 0; i < superBlocks.length; i++) {
if (superBlocks[i]['dashedName'].toString().contains('full-stack')) {
continue;
}
// if (superBlocks[i]['dashedName'].toString().contains('full-stack')) {
// continue;
// }
Comment on lines +246 to +248
Copy link
Member

@huyenltnguyen huyenltnguyen Oct 20, 2025

Choose a reason for hiding this comment

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

Do we have a way to enable FSD for dev only, without making it available in prod?

Copy link
Member Author

@Sembauke Sembauke Oct 20, 2025

Choose a reason for hiding this comment

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

yep this is the way

Copy link
Member Author

@Sembauke Sembauke Oct 23, 2025

Choose a reason for hiding this comment

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

I think this is mostly a relic from what we used to do with the new landing page for the full stack cert

layout.add(
SuperBlockButton(
button: SuperBlockButtonData(
Expand Down
Loading