Skip to content

Conversation

@OmkarPh
Copy link
Collaborator

@OmkarPh OmkarPh commented May 14, 2023

Part of #536
Also fixes #576

OmkarPh added 4 commits May 9, 2023 10:52
Signed-off-by: Omkar Phansopkar <omkarphansopkar@gmail.com>
…pe defs

Signed-off-by: Omkar Phansopkar <omkarphansopkar@gmail.com>
Signed-off-by: Omkar Phansopkar <omkarphansopkar@gmail.com>
Signed-off-by: Omkar Phansopkar <omkarphansopkar@gmail.com>
@OmkarPh OmkarPh changed the base branch from develop to v4.0-react-typescript May 14, 2023 17:19
OmkarPh added 2 commits May 15, 2023 22:09
Signed-off-by: Omkar Phansopkar <omkarphansopkar@gmail.com>
Signed-off-by: Omkar Phansopkar <omkarphansopkar@gmail.com>
Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

@OmkarPh Thanks+++ Looks great! Both File->package view and File->License view works perfectly. Putting them here for reference.
packages-links
license-links

…ted in #576

Signed-off-by: Omkar Phansopkar <omkarphansopkar@gmail.com>
OmkarPh added 2 commits June 2, 2023 19:48
Signed-off-by: Omkar Phansopkar <omkarphansopkar@gmail.com>
Signed-off-by: Omkar Phansopkar <omkarphansopkar@gmail.com>
Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

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

LGTM! @OmkarPh .
Looks great, go ahead and merge!
I missed in the previous review, the Package -> file, License -> File links works nicely too.
This feature adds a lot of convenience ❤️

@OmkarPh
Copy link
Collaborator Author

OmkarPh commented Jun 8, 2023

thanks, merging

@OmkarPh OmkarPh merged commit c53662c into v4.0-react-typescript Jun 8, 2023
@AyanSinhaMahapatra AyanSinhaMahapatra deleted the feature/goToFilePath branch October 25, 2023 18:55
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