Skip to content

Fix file paths mentioned in website's Contribute page#1052

Merged
Keavon merged 6 commits into
GraphiteEditor:masterfrom
tlaplaca:fix-paths-website-contribute-page
Feb 23, 2023
Merged

Fix file paths mentioned in website's Contribute page#1052
Keavon merged 6 commits into
GraphiteEditor:masterfrom
tlaplaca:fix-paths-website-contribute-page

Conversation

@tlaplaca

Copy link
Copy Markdown
Contributor

File path updates

Rust translation layer:
image

Editor FrontendMessage handler:
image

Rust to JS-friendly data types mapping:
image

Comment thread website/content/contribute/_index.md Outdated

@Keavon Keavon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for noticing and fixing this.

@Keavon Keavon changed the title Updated paths for frontend / backend communication section of contribute page Fix file paths mentioned in website's Contribute page Feb 23, 2023
@Keavon Keavon merged commit 6d43d8d into GraphiteEditor:master Feb 23, 2023
@tlaplaca tlaplaca deleted the fix-paths-website-contribute-page branch March 3, 2023 06:27
Keavon added a commit that referenced this pull request Jul 30, 2023
* Updated paths for frontend / backend communication section of contribute page to reflect current project structure

* Added starting / for consistency with other paths on the page

* Mapping actually happens in subscription router

* Changed path to show code that dispatches FrontendMessage rather than the type def

* Missing / at start of path

* Fix files

---------

Co-authored-by: Keavon Chambers <keavon@keavon.com>
@Keavon

Keavon commented Jun 13, 2026

Copy link
Copy Markdown
Member

Hi @tlaplaca, thanks again for this code contribution to the project! We're still hoping you will respond to the request to relicense this code. Please see #4208 ASAP, thank you!

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.

2 participants