Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Mar 9, 2021. It is now read-only.

Remove user login and profile functionality #24

Merged
merged 23 commits into from
Oct 19, 2017
Merged

Remove user login and profile functionality #24

merged 23 commits into from
Oct 19, 2017

Conversation

vinaypuppal
Copy link
Contributor

@vinaypuppal vinaypuppal commented Oct 18, 2017

Changes

  • Removed all backend dependent code

  • Removed login, profile, jobs, projects pages

  • Fixed landing page image load flickering issue(i.e footer jumps from header to bottom after image loading)

  • Removed axios and used browser fetch API and as a fallback for older browsers added unfetch polyfill

  • Removed moment.js and used a light weight alternative to it called date-fns

    App size comparisons before and after
  • Before (256.83KB) screen shot 2017-10-19 at 01 55 02
  • After (Now its 127.68KB) 🎉 screen shot 2017-10-19 at 01 55 02
  • Visually separate upcoming and past events.

    Screen Shot screen shot 2017-10-19 at 01 55 02

  • Modified app to use next.js satic export functionality so we can deploy app as static app since we don't have any backend dependent code.

  • Modified travis script to automatically deploy a unique app for every pull req and deploy to https://coderplex.org when merged to master.

  • Added support for @coderplex-bot so it will comment with new deployment url for every PR or PR changes. 🎉

@coderplex-bot
Copy link
Collaborator

Δ Now staging deployment complete


Note: This is autogenerated through Travis build

@vinaypuppal
Copy link
Contributor Author

Squash and Merge because there are lot of test commits which I used for testing in travis-ci

@coderplex-bot
Copy link
Collaborator

New Δ Now staging deployment complete


Note: This is autogenerated through Travis build

@M-ZubairAhmed M-ZubairAhmed merged commit ec75b42 into master Oct 19, 2017
@M-ZubairAhmed M-ZubairAhmed deleted the travis branch October 19, 2017 13:11
Copy link

@M-ZubairAhmed M-ZubairAhmed left a comment

Choose a reason for hiding this comment

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

all good @vinaypuppal if its by you

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants