Skip to content

Conversation

@saumyashahi
Copy link
Contributor

Description:

This PR adds the Week 02 progress report blog post for GSoC '25 by Saumya Shahi. The post includes:

  • Summary of goals, accomplishments, learnings, and challenges during the second week of GSoC.
  • Rendered SVG brick screenshots demonstrating development progress on the Masonry Module.
  • Associated image assets for inclusion in the blog post.

Changes Made:

  • Added markdown file for blog post.
  • Added images used in the post to: public/assets/images

Testing Performed:

  • Verified that all image paths are correctly linked and render as expected.
  • Previewed the blog post layout in local development environment.

Checklist

  • I have tested these changes locally and they work as expected.
  • I have added/updated tests that prove the effectiveness of these changes.
  • I have updated the documentation to reflect these changes, if applicable.
  • I have followed the project's coding style guidelines.
  • I have addressed the code review feedback from the previous submission, if applicable.

@github-actions
Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

Copy link
Contributor

@pikurasa pikurasa left a comment

Choose a reason for hiding this comment

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

Please see comment.

There may be other things, please run your code and test it.

2. **Rendered Output Validation & Screenshots**
- Verified each visual brick against expected path geometry.
- Screenshots below show rendered bricks:
- ![Simple Brick](public\assets\Images\simple-bricks.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

These don't seem to be rendering properly.

Screenshot from 2025-06-16 12-46-38

Please look into this, make a fix, and test your code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay I will check and fix this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have fixed it, it should render fine now !

@pikurasa pikurasa marked this pull request as draft June 16, 2025 16:50
@github-actions
Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@saumyashahi saumyashahi marked this pull request as ready for review June 16, 2025 17:11
@pikurasa pikurasa merged commit 8f69b80 into sugarlabs:main Jun 16, 2025
3 checks passed
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