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

Release 1.2.13 #56

Merged
merged 6 commits into from Sep 29, 2022
Merged

Release 1.2.13 #56

merged 6 commits into from Sep 29, 2022

Conversation

audreylamy
Copy link
Contributor

  • Nouveau nom d'extension
  • Nouveau logo
  • Modification manifest chrome et firefox
  • Modification package.json

fixes #50

@audreylamy audreylamy self-assigned this Sep 23, 2022
Copy link
Collaborator

@bt0r bt0r left a comment

Choose a reason for hiding this comment

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

Juste le bump de version à faire et c'est good pour moi

@@ -1,8 +1,8 @@
{
"manifest_version": 2,
"name": "NoteFlix",
"name": "SensCritique",
"version": "1.2.11",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Upgrade la version en respectant semver :)

@@ -1,8 +1,8 @@
{
"manifest_version": 2,
"name": "NoteFlix",
"name": "SensCritique",
"version": "1.2.11",
Copy link
Collaborator

Choose a reason for hiding this comment

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

idem

@@ -18,7 +18,6 @@
}
],
"permissions": [
"*://www.allocine.fr/*",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Par contre si tu supprimes la permission à Allociné il faut supprimer le code aussi (sinon l'extension ne fonctionnera pas en l'état)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Edit: tu as fait une autre PR pour ca donc tu peux rollback cette partie là et l'intégrer à l'autre PR (comme ca cette release reste fonctionnelle)

@audreylamy audreylamy merged commit f62cd8e into master Sep 29, 2022
@audreylamy audreylamy deleted the fix/new-name-logo branch September 29, 2022 15:17
@audreylamy audreylamy changed the title fix(naming): nouveau nom d'extension et ajout nouveau logo release 1.2.13 Sep 29, 2022
@audreylamy audreylamy changed the title release 1.2.13 Release 1.2.13 Sep 29, 2022
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.

Noteflix: changer nom + logo
2 participants