iiitv / getmein-web

Portal to get an invite to IIITV GitHub organization

getmein-web

Getting into IIITV Open Source Organization just got easier!

How to Use

Go to http://getmein.glitch.me to get invitations to join this GitHub organization and the Slack Workspace.

Previously, joining the IIITV Organization was a bit tedious - create an issue in the iiitv/getmein repository and wait for the invite.

Now, enter your GitHub username and E-mail ID, get in easily, and start contributing!

Show Yourself a part of IIITV

Your GitHub profile won't show you as a part of IIITV's organization. To make your membership public, follow this article.

Next Steps

• Join us on Slack.
• Add yourself to Members team if not already a member of it.
• Join the team for your batch (batch-of-20XX).
• Solve issues in organisation's repositories.
• Create your project and collaborate with others on Gitter Rooms and Slack Channels.

Do not belong to IIITV

No problem, you are still welcome to our Outsiders team.

For contributors

CONTRIBUTING.md

To install the dependencies

npm install

To start the local server

npm start

Configuration

You can configure this application to use with your environment specified settings. Just copy these Environment variables to .env and make necessary changes. Don't commit this .env file or remove it from .gitignore .

Environment variables

VAR	DESCRIPTION	VALUES	DEFAULT
SECRET	Your GitHub Token	-	-
GMAIL_KEY	Gmail token for email	-	-

Stargazers over time📈

Made With ❤ By IIITV Coding Club

getmein-web's Issues

Add .editorconfig

Add a .editorconfig file.

Footer containing links overlaps with body on small screens

Expected Behavior

The footer containing the links should appear below the body. If there is less screen space, it should appear below after scrolling and not on top of it.

Current Behavior

On small screens and in landscape mode, this overlapping happens.

Your Environment

On most mobile browsers in landscape mode or while typing.

Would you like to work on this issue?

No

[Feature] Add ESLint

Expected Behavior

You have to add a default linter. Preferable - ESLint

You can take help from #38

Page doesn't open in new tab

Expected Behavior

when we click on the social icons the page should open in a new tab
It would be better if the page opens in a new tab

Current Behavior and Triggers

when we click on the social icons the page doesn't open in a new tab
The user experience would be better if the page opens in a new tab

Possible Solution

The above issue can be fixed by adding the target attribute ;
adding target attribute fixes this issue

Would you like to work on this issue?

Yes

Improve README

Use HTTP Authorisation

The GitHub team sent this reminder about the deprecation issue.

Please use the Authorization HTTP header instead, as using the access_token query parameter is deprecated. If this token is being used by an app you don't have control over, be aware that it may stop working as a result of this deprecation.
Depending on your API usage, we'll be sending you this email reminder on a monthly basis for each token and User-Agent used in API calls made on your behalf.
Just one URL that was accessed with a token and User-Agent combination will be listed in the email reminder, not all.
Visit https://developer.github.com/changes/2020-02-10-deprecating-auth-through-query-param for more information about suggested workarounds and removal dates.

Validate Email Function

getmein-web/public/client.js

Lines 1 to 6 in e3c7292

	// @todo: Validate Email Function
	// $('#email').keyup(function () {
	// const email = $('#email')
	// if (email.val()) {
	// if (validateEmail(email.val())) {

---

This issue was generated by todo based on a @todo comment in e3c7292 when #73 was merged. cc @iiitv.

Display a message for students to use Institute ID

Accept email of the form [email protected] or else display a prompt that asks new users to use only Institute ID.

Add a code linter

Add coala or similar linter for JS files.
Anyone from @iiitv/batch-of-2021, please take this issue.

Refrences: https://codepen.io/jesperkc/pen/pydMqb

Originally posted by @SasiniSithara in #19

add@omkumarbhu123

Expected Behavior

Current Behavior and Triggers

Possible Solution

Your Environment

• Version used:
• Environment name and version (e.g. Chrome 39, node.js 5.4):
• Operating System and version (desktop or mobile):
• Link to your project:

Would you like to work on this issue?

Add pre-hook commit to force eslint rules

Expected Behavior

Whenever someone wants to do some commit eslint should run so
that we have consistent coding style

Current Behavior and Triggers

no enforcing of style guidelines

Possible Solution

Your Environment

• Version used:
• Environment name and version (e.g. Chrome 39, node.js 5.4):
• Operating System and version (desktop or mobile):
• Link to your project:

Would you like to work on this issue?

Auto-update Glitch with GitHub

Resources: https://support.glitch.com/t/tutorial-how-to-auto-update-your-project-with-github/8124

Prompt to Check Inbox for Invite

Expected Behavior

After a successful request user should get a prompt to wait and check for email as some users keep on requesting again.

New message Thank You, Check your inbox for the invite

Current Behavior and Triggers

Currently, the user is getting await

Possible Solution

Here we can change the message to Thank You, Check your inbox for the invite
and uk-spinner to uk-icon='icon: check; ratio: 2'

Your Environment

N/A

Would you like to work on this issue?

Yes

Add Dark/Light Mode

Add one toggle to change color scheme

Problem with no scrolling in phones

Expected Behavior

The whole page should be visible on all devices.

Current Behavior and Triggers

Page gets trimmed for small screens.

Possible Solution

Enable scroll again.

Would you like to work on this issue?

No

Move to API v3 version to update membership

Currently, we use org_id and team_id to update membership which was the only way when this project was created, the new API v3 enables using team slug and org username for the same/
Ref: https://developer.github.com/v3/teams/members/#add-or-update-team-membership-for-a-user

PUT /orgs/:org/teams/:team_slug/memberships/:username

Fix Lint in windows

Expected Behavior

On windows npm run lint gives an error.
So we can't commit and push as we husky will give the error again.

Possible Solution

One solution is to add another command for windows in package.json file
like "lint-win": "eslint ./public/*.js *.js --fix"
and use or in pre-commit so that any one of them will run.

Your Environment

• Version used:
  Windows 10

Would you like to work on this issue?

No

Update README

Add detail about the project in README.

Add Instruction to take a look at Code of Conduct and Contribution Guidelines

Add Instruction to take a look at Code of Conduct and Contribution Guidelines on getmein web page, email sent to new member and README too.

Fix instant username check

Currently, the function checks if the username exists or not, but there are some issues with function.

Also, we have to add classes to the input box,
i.e. on receiving 200: addClass('uk-form-success') and on 404: addClass('uk-form-danger')

getmein-web/public/client.js

Lines 9 to 27 in 0240d97

	function ghAccountExists(username) {
	let profile = "https://api.github.com/users/" + username;
	if(getStatus(profile) == 200) {
	return true;
	}
	return false;
	}
	function getStatus(url) {
	var request = new XMLHttpRequest();
	request.onreadystatechange = function() {
	if (request.readyState === 2){
	return request.status;
	//this contains the status code
	}
	};
	request.open("GET", url, true);
	request.send();
	}

Remove IIITVians from outsider

If someone signs up using iiitv email, remove them from the outsider team, just in case they have messed up earlier.

Fix Remove from only outsider

Issue when toggling Dark/Light themes

Expected Behavior

  The font in the textbox, when toggling the website from Black Theme to white theme, should be visible

Current Behavior and Triggers

It shows white font, thus on toggling , the font is not visible, unless we edit the data.

Possible Solution

if we could adjust the font-color on toggling the themes.

Your Environment

• Version used:
• Environment name and version (e.g. Chrome 39, node.js 5.4):Chrome 74.0.3729.169 (Official Build) (64-bit
• Operating System and version (desktop or mobile): Windows 10 x64, Desktop
• Link to your project:

Would you like to work on this issue?

No

Images, showing this issue, are attached

Update createmail.js's Styles

Currently, the images are not sized properly.
Send yourself an email and check for the issue.

I would Like This Organization

Link provided only sent slack Invitation.
Please Add Me To Your Organization
[email protected]
@JayantGoel001

Always shows "invalid username"

@rossoskull you commit which merged username check, it somehow always shows the toast when we move the cursor over avatar icon.

Fix the verify email part

Current verify email throws an error when calling GitHub API.
This can be done simply by replacing slug by id for team

Invite to Slack

If the email address is from IIITV, invite the member to Slack too.

Resource: https://api.slack.com/methods/channels.invite

Add a PR Template

Add PR template to help reviewers to know which issue is referenced and what changes are done with a demo.

Add Issue Template

Add an issue template, helping people to elaborate issues more.

CORS error

Expected Behavior

All the API's call should be working.

Current Behavior and Triggers

Many features are blocked due to CORS error in API call.

Possible Solution

I tried many soln, it did'nt work at all, i also tried stackoverflow soln but it also did'nt worked.
Adding this link https://cors-anywhere.herokuapp.com/ before all the api links will work but it will make app too slow.
Can you suggest any other soln.

Would you like to work on this issue?

Yes

Screenshot of issue

Improve UI

Expected Behavior

at first look page looks like this is it , no need to scroll down . but only if we scroll we get to see social links .

Possible Solution

So, we can give a dancing down arrow which attracts users to social links or we can do some changes in UI and made links available at left side or just move them up . um like slack invitation page

I can work on this issue.

Fix Body Height

The current page's body height is not fit to screen.
Change it to not scrollable one.

add nishant-madara

Expected Behavior

Current Behavior and Triggers

Possible Solution

Your Environment

• Version used:
• Environment name and version (e.g. Chrome 39, node.js 5.4):
• Operating System and version (desktop or mobile):
• Link to your project:

Would you like to work on this issue?

No scroll on small screens

Expected Behavior

On screens with height big enough, all the content is displayed so there's no need to scroll but on small screens, scrolling is needed to see all the content.

Current Behavior and Triggers

No scrollbar on small screens
Might be related to #70 and #78

On Landscape Mode in Android

In small Window

Your Environment

Tested on Edge (Chromium Dev and Default) on Windows, and Chrome on Android

Would you like to work on this issue?

No

GitHub Invites not working

Update deprecated API URL and fix the AXIOS PUT request.

Wrong Username

When I enter a wrong GitHub username instead of error it send a verification mail.

Send message to slack when someone gets invited

When sending an email, also post to slack.
You can access the webhook URL using process.env.INVITE_CHANNEL_WEBHOOK and try to post on your own test channels maybe.

Reference: https://api.slack.com/incoming-webhooks#posting_with_webhooks

Integrate Node Mailer

setup GitHub actions

Add Travis CI

Add Travis CI, so that we can check for Linit and other errors in CI's environment itself.

Git Sync with Glitch is failing

Expected Behavior

On merge to master, deployment on glitch should auto pull changes.

Current Behavior and Triggers

On merge, the post request on /git fails with error in console.

Possible Solution

The current setup for auto-sync to glitch on merge is messed up, we might need to move to a better approach maybe.
PS: I have a similar old set up from a private project, will share that approach if needed.

Your Environment

N/A

Would you like to work on this issue?

No

Add Email Template

Add email template and encrypt the invite link into it.
On clicking the link, it should decrypt and proceed to team pages of iiitv.

Fix tooltips of footer social links

Expected Behavior

Decrease the offset of the tooltip so they appear near the icons.

Current Behavior and Triggers

Tooltips of the footer social link icons seem to have a high value of offset.

Would you like to work on this issue?

No.

preload stylesheet and async script loading for improve page load time

Refactor Codebase

When the project started, the project did well with hardcoded values like team_ids but now with the teams increasing, we should fetch them from API.

A refactoring overhaul is much needed.

Tip: Use principles taught in CS303 Course 😉

Add Tests, Coverage

Expected Behavior

Good coverage

Current Behavior and Triggers

No tests, evil

Possible Solution

Setup tests, run in CI on every PR, preferably in actions

Your Environment

Not needed

Would you like to work on this issue?

No

Read transport config from env

Refer -

getmein-web/server.js

Line 50 in f6800f6

host: 'smtp.gmail.com',

Configure SendGrid for sending mails