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

dead_code : Delete openlibrary/templates/contact/spam/sent.html #9566

Merged
merged 2 commits into from
Jul 15, 2024

Conversation

harsh-kumar-patwa
Copy link
Contributor

Closes #9174

This PR deletes the dead code "openlibrary/templates/contact/spam/sent.html".

Technical

After investing about the code present in "openlibrary/templates/contact/spam/sent.html", I could conclude that it was not being used anywhere in the whole repositry. So I have removed the file and have updated the exclude list also.

Testing

Screenshot

Stakeholders

@mekarpeles

@rebecca-shoptaw
Copy link
Collaborator

Hi @harsh-kumar-patwa & @mekarpeles! I wanted to give you both a heads-up that this has already been fixed in #9505, which @cdrini is currently reviewing.

I'd rather keep the fix there if possible to keep all the i18n updates a result of the script nicely batched, but if you'd prefer to do it separately, I can remove the fix from that PR! Just let me know what you prefer 🙂

@cdrini cdrini assigned scottbarnes and unassigned mekarpeles Jul 15, 2024
Copy link
Collaborator

@scottbarnes scottbarnes left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for this, @harsh-kumar-patwa.

@rebecca-shoptaw, during the weekly OL core team meeting we decided to handle this one separately, which is why it is getting merged. Sorry for the need to rebase this causes. :(

@scottbarnes scottbarnes merged commit 5aa8c6e into internetarchive:master Jul 15, 2024
4 checks passed
@cdrini
Copy link
Collaborator

cdrini commented Jul 15, 2024

To follow up, I've been having some issues testing #9505 , so I think it'll take a few days to sort out, so didn't want to block this one!

@rebecca-shoptaw
Copy link
Collaborator

@scottbarnes @cdrini All good!! Rebasing has become basically one of my hobbies with all the i18n and registration PRs, so I don't at all mind doing a little more 🙂

Glad this is getting out, and excited to continue with the next i18n step as time/testing allows!

@harsh-kumar-patwa
Copy link
Contributor Author

LGTM. Thanks for this, @harsh-kumar-patwa.

@rebecca-shoptaw, during the weekly OL core team meeting we decided to handle this one separately, which is why it is getting merged. Sorry for the need to rebase this causes. :(

No problem ! It was was my first pr getting merged as I has started my open source journey recenlty. It's a great achievement for me and motivates me to continue contributing further.
Thanks to whole openlibrary team.

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

Successfully merging this pull request may close these issues.

Is openlibrary/templates/contact/spam/sent.html dead code?
5 participants