Skip to content
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.

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

Updated the people.html file with GCE2 registrants information #10

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

SaptSinha
Copy link
Contributor

Name and Institution (Required)

Name: Saptarshi Sinha
Institution: Washington University in St. Louis

Confirm you have reviewed the following documentation

Describe the update

I have updated the people.html file with the information obtained from the GCE2 registrants' database. Some entries were not retrievable. I will provide an excel sheet that shows the entries that were successfully updated, and the ones that weren't.

Expected changes

This update will populate the people.html page with the latest information.
NOTE: I will push another update from the email list as well in this week.

Related Github Issue(s)

Not applicable.

@msulprizio msulprizio self-requested a review July 17, 2023 18:15
@msulprizio msulprizio self-assigned this Jul 17, 2023
Copy link
Contributor

@msulprizio msulprizio left a comment

Choose a reason for hiding this comment

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

These changes look good to merge. Thanks @SaptSinha!

@msulprizio msulprizio merged commit 0cfb8e7 into geoschem:main Jul 17, 2023
@lizziel
Copy link

lizziel commented Jul 17, 2023

It looks like the indentations of some of the later parts are bit random. Do we have a standard tool to run on people.html to make the indentation consistent?

@msulprizio
Copy link
Contributor

@lizziel No. The indentation is somewhat of a mess in this file even before these updates. In the past, I manually edited it but gave up because emacs automatically idents occasionally when adding spaces or new lines.

Copy link
Contributor

@msulprizio, @lizziel, There are some online tools that might help, such as: https://www.freeformatter.com/html-formatter.html

@msulprizio
Copy link
Contributor

Thanks @yantosca @lizziel. I've opened a new issue at #12 to track the progress on this request.

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

Successfully merging this pull request may close these issues.

4 participants