Skip to content

Conversation

@Bishoywadea
Copy link
Contributor

Description

Report for week 3.

Changes Made

Added 2025-06-22-gsoc-25-BishoyWadea-week03.md-> blog report

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 fix the URL (by removing slug), and please consider adding an image or two.

excerpt: "Broken Calculator"
category: "DEVELOPER NEWS"
date: "2025-06-22"
slug: "gsoc-25-BishoyWadea-week03"
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't believe you need the slug field. @sa-fw-an can confirm.

Also, by removing it, I expect that the URL will default to a naming convention consistent with what we have elsewhere.

Copy link
Member

Choose a reason for hiding this comment

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

It is needed, it basically decides the url of the blog, if that is removed it will automatically take the filename as the url endpoint


## This Week’s Achievements

### *Goal 1: Fix Issues in Four Color Map Activity*
Copy link
Contributor

Choose a reason for hiding this comment

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

I recommend adding an image or two. This will help readers who are less familiar with your work to better understand what you're working on.

I know some images are on the pages you link to, but embedding them to the page will probably also help people get excited enough to "read on" and check out those links.

@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. 👏


- **Goal 1:** Fix issues in Four Color Map game opened by ibiam [Move buttons from canvas to tool bar](https://github.com/Bishoywadea/Four-Color-Map/issues/1), [Sugarize activity icon](https://github.com/Bishoywadea/Four-Color-Map/issues/2), [adding lisence to activity](https://github.com/Bishoywadea/Four-Color-Map/issues/3).

![b4e81f67-3e05-4389-a67d-82eb71a14899.jpg](https://i.postimg.cc/vmWPFd6q/b4e81f67-3e05-4389-a67d-82eb71a14899.jpg)
Copy link
Contributor

Choose a reason for hiding this comment

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

The text in between the brackets become the caption for the image.

Screenshot from 2025-06-23 12-50-27

Please create a meaningful description.

@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. 👏

@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. 👏

@Bishoywadea
Copy link
Contributor Author

@sa-fw-an @pikurasa I have added this week's blog to this PR as well for review. If you'd prefer it to be in a separate PR, just let me know.

@pikurasa
Copy link
Contributor

pikurasa commented Jul 1, 2025

@Bishoywadea There are errors, but I will fix them. Otherwise the report is good. ...merging

@pikurasa pikurasa merged commit 4dbc16b into sugarlabs:main Jul 1, 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.

3 participants