Technical Contributions to Telescope — Weeks 2–3
In the last couple of weeks, we’ve focused on cleaning up the PRs section at Telescope and continued porting to Next.js, along with some improvements to developer experience, theme, and CI/CD.
Here is a list of PRs that I have reviewed:
- https://github.com/Seneca-CDOT/telescope/pull/1483 — Fixes #1386: set up Theme src/frontend/src/theme.js in Next
- https://github.com/Seneca-CDOT/telescope/pull/1487 — [next] Fixes #1463 — re-implements the About page w/ nextjs
- https://github.com/Seneca-CDOT/telescope/pull/1505 — Fixed #1491: Port MyFeedsPage, see src/frontend/src/pages/myfeeds.js to NextJS
- https://github.com/Seneca-CDOT/telescope/pull/1562 — [next] Item 2 and 4 related to /styles/global.css of Issue #1548
- https://github.com/Seneca-CDOT/telescope/pull/1566 — Fixed #1508: Ported SEO component to NextJS
- https://github.com/Seneca-CDOT/telescope/pull/1568 — Fixed #1456: Ported Login component to NextJS
- https://github.com/Seneca-CDOT/telescope/pull/1571 — [next] — Add Feed types to list of interfaces #1570
- https://github.com/Seneca-CDOT/telescope/pull/1574 — Providing Theme object to the whole app
- https://github.com/Seneca-CDOT/telescope/pull/1587 — Update README with Netlify badge for next.js builds
- https://github.com/Seneca-CDOT/telescope/pull/1591 — Fix #1589 — make sure that NEXT_PUBLIC_API_URL is set locally and in Netlify
- https://github.com/Seneca-CDOT/telescope/pull/1615 — [next] Fixes #1544: Porting ExistingFeedList to Next.js
- https://github.com/Seneca-CDOT/telescope/pull/1619 — fix #1494: Port adminbutton to [next]
- https://github.com/Seneca-CDOT/telescope/pull/1623 — Allow proxying next’s webpack dev server (add unsafe-eval CSP for script-src) via backend
- https://github.com/Seneca-CDOT/telescope/pull/1624 — [next] Fixes #1463 — re-re-implements the About page w/ nextjs
- https://github.com/Seneca-CDOT/telescope/pull/1631 — Fix #1629: improve next.js auth handling
- https://github.com/Seneca-CDOT/telescope/pull/1639 — linter: remove ‘Prefer async/await to callback’ rule from .eslintrc
My Technical Contribution
As for my own contribution, I have focused on picking up the PR originally opened by Andy last semester, which has introduced automated release workflow with GitHub Actions. I have improved it to add tests and use the correct tag regex. The updated and rebased script was successfully merged in https://github.com/Seneca-CDOT/telescope/pull/1628 to close issue #1301. You can read more about workflow automation with GitHub Action in the dedicated article.
I really enjoyed learning more about CI/CD and the help of our team members played a big role in it. Special thanks to Josue for spending a couple of nights discussing the workflow with me!
There is still room for improvements (issue #1632 and issue #1638), which I am planning to hop on in the upcoming days.