Skip to content

Develop #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
orama254 merged 58 commits into from
Aug 19, 2022
Merged

Develop #71

orama254 merged 58 commits into from
Aug 19, 2022

Conversation

@orama254
Copy link
Member

@orama254 orama254 commented Aug 19, 2022

Fixes Issue

Major codebase change with latest changes from July Sprint

Changes proposed

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

stephanieopala and others added 30 commits June 14, 2022 11:23
@stephanieopala
Merge pull request #41 from stephanieopala/develop
b335572 
Update develop branch
@AJ-Kulundu
Website link fix in the readme file
d7dc198
@mk-mukul
create the navigation bar [Fixes #32]
26f1d94
@zidanegimiga
style: add montserrat font
575e789 
- create a custom document page to fetch fonts on the server-side from Google fonts CDN.
- transferred meta tags to document page.
@zidanegimiga
refactor: formatted code with Prettier
f75a6f6
@nyandika
Pin dependencies
c5ee124
@nyandika
Use playwright for e2e tests
917fdb2
@orama254
Merge pull request #45 from AJ-Kulundu/website-link-fix
9c7b9aa 
Website link fix in the readme file
@orama254
Merge pull request #46 from mk-mukul/new_branch
f3434ad 
create the navigation bar [Fixes #32]
@orama254
Merge pull request #47 from zidanegimiga/adding-fonts
b987238 
Adds fonts from Google
@orama254
Merge pull request #48 from nyandika/feature/e2e
033fb5f 
Add e2e tests runner using playwright
@orama254
fix(project): cleanup and aesthetics
f2a5b00
@orama254
feat(gh-actions): add package-lock file to facilitate playwright gith…
ff69b9f 
…ub action
@orama254
Merge pull request #49 from orama254/develop
58a4cc5 
minor code cleanup
@AJ-Kulundu
Accepting bit.ly placehoder image
c361647
@AJ-Kulundu
Setting up aspect ratio for responsive image sizes
5b4723d
@AJ-Kulundu
The reusable Image component
836c419
@AJ-Kulundu
The events page which shows multiple events
2af8926
@lekipising
create pages
cbc6705
@lekipising
update navbar links to point to pages
66e241c
@lekipising
create routes constant file
3ba777f
@lekipising
convert js to jsx and update pages H1
1979aa6
@lekipising
convert app and document files to jsx
6cc352a
@lekipising
add title in the pages
df2f326
@AJ-Kulundu
Changing the name of the reusable component
88576b4
@AJ-Kulundu
Merge branch 'develop' into events-page
a169f69
@orama254
Merge pull request #53 from Lekipising/feat-add-pages
28b2d54 
Add pages, convert files to jsx and create global routes file [Fixes #52]
@orama254
refactor(navbar): add join group link and logo to navbar
04f3aae
@orama254
Merge pull request #54 from orama254/develop
2d96591 
refactor(navbar): add join group link and logo to navbar
@AJ-Kulundu
sorted out merge conflicts
6073ea2
AJ-Kulundu and others added 27 commits June 16, 2022 18:44
@AJ-Kulundu
The events page which shows multiple events
38357e5
@AJ-Kulundu
Merge branch 'reactdeveloperske:main' into events-page
8becd01
Merge remote-tracking branch 'upstream/develop' into develop
4da7518
updated contribution doc landing page styling [Fixes #57]
3ebdde4
review updated
32fc384
remove changes to codebase
2b76f5e
remove changes to codebase
c4bb82f
remove changes to codebase
7f4822a
@AJ-Kulundu
Fixing the aspect-ratio plugin and moving the helper function into th…
7ba3f32 
…e util folder
[add] banner
de8d9d4
[add] Banner component improved [Fixes #51]
6dfae73
modified Banner [Fixes #51]
13f0225
@orama254
Merge pull request #59 from dmuyah/57@muyah
002f504 
57@muyah
@orama254
Merge pull request #63 from bilalscripts/develop
fab94d9 
[add] Banner Component [Fixes #51]
@orama254
Merge pull request #50 from AJ-Kulundu/events-page
19cb2f9 
Events page and events reusable component [Fixes #43]
@orama254
refactor(issue#64): add modifications based on folder structure discu…
4234e2f 
…ssion
@orama254
fix(runtime-error): add fix to sort runtime error
db7674d
@orama254
fix(issue#64): add fix to resolve folder structure conflicts
fbf79b4
@orama254
feat(page-titles): add descriptive titles for SEO
594c1e0
@orama254
fix(playright): add new test directory
a69a864
@orama254
refactor(issue#65): add modifications based on folder structure discu…
b063fdd 
…ssion
@orama254
fix(api-directory): refactor to use new api extension
5b71795
@orama254
Merge pull request #66 from orama254/develop
62e270f 
Change folder structure based on discussion
@orama254
chore(ci):linter rules applied to codebase
058a902
@orama254
chore(ci):lint and prettier configuration
b535438
@orama254
chore(ci): add changes based on code review
9f20a3a
@orama254
Merge pull request #70 from orama254/develop
21651ec 
backlog task: add eslint and prettier to the project
@vercel
Copy link

vercel bot commented Aug 19, 2022 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
reactdevske-website ✅ Ready (Inspect) Visit Preview Aug 19, 2022 at 11:32AM (UTC)

@netlify
Copy link

netlify bot commented Aug 19, 2022 •
edited
Loading

✅ Deploy Preview for reactdevske ready!

Name Link
🔨 Latest commit 21651ec
🔍 Latest deploy log https://app.netlify.com/sites/reactdevske/deploys/62ff74bc3f825c0008929eb9
😎 Deploy Preview https://deploy-preview-71--reactdevske.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@orama254 orama254 merged commit 56deb45 into main Aug 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@orama254 @stephanieopala @AJ-Kulundu @mk-mukul @zidanegimiga @nyandika @lekipising