alexeyinkin commented on code in PR #22158:
URL: https://github.com/apache/beam/pull/22158#discussion_r913700768
##########
playground/frontend/lib/pages/routes.dart:
##########
@@ -17,29 +17,35 @@
*/
import 'package:flutter/material.dart';
-import
'package:playground/pages/embedded_playground/embedded_page_providers.dart';
+import 'package:playground/constants/params.dart';
+import
'package:playground/pages/embedded_playground/embedded_playground_page.dart';
import 'package:playground/pages/playground/playground_page.dart';
class Routes {
- static const String playground = '/';
static const String embedded = '/embedded';
static Route<dynamic> generateRoute(RouteSettings settings) {
final name = settings.name ?? '';
final queryIndex = name.indexOf('?');
final routePath =
name.substring(0, queryIndex < 0 ? name.length : queryIndex);
+
switch (routePath) {
- case Routes.playground:
- return Routes.renderRoute(const PlaygroundPage());
Review Comment:
It is the default route, so no need to handle it separately.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
