Skip to content
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.

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

feat: add new roadmap page #388

Merged
merged 3 commits into from
Jul 19, 2024
Merged

feat: add new roadmap page #388

merged 3 commits into from
Jul 19, 2024

Conversation

melaniebmn
Copy link
Collaborator

Summary | Résumé

Add a new roadmap page.

Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-388.d35vdwuoev573o.amplifyapp.com

Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-388.djtlis5vpn8jd.amplifyapp.com

adorayi added 2 commits July 19, 2024 08:33
Added in EN text, copied from the GC Design System Roadmap document
Added FR text from roadmap document.
@adorayi adorayi marked this pull request as ready for review July 19, 2024 15:02
Copy link
Collaborator

@ethanWallace ethanWallace left a comment

Choose a reason for hiding this comment

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

LGTM!

@daine
Copy link
Collaborator

daine commented Jul 19, 2024

LGTM! @adorayi are we linking to this page from somewhere else in the site?

### Theme: Product Growth

#### Items:
- **Support for Server Side Rendering frameworks**
Copy link
Collaborator

Choose a reason for hiding this comment

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

@adorayi oh just noticed, should be mention that it's currently specific to React?

Suggested change
- **Support for Server Side Rendering frameworks**
- **Support for React with Server Side Rendering frameworks**

Copy link
Contributor

Choose a reason for hiding this comment

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

That was more of a general statement which the sub bullet expands on with details (NextJs). I don't think we need to change this now, we can revisit later. Thanks.

@adorayi
Copy link
Contributor

adorayi commented Jul 19, 2024

LGTM! @adorayi are we linking to this page from somewhere else in the site?

No need to link it for now, we can share the direct url for now and then consider the full site structure another time.

@adorayi adorayi merged commit 21dbe5d into main Jul 19, 2024
3 checks passed
@adorayi adorayi deleted the add-roadmap-page branch July 19, 2024 18:33
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.

4 participants