Update common-interfaces.ts

[nicoraffin]

See issue #11.
Optional interface elements are blocking TypeScript compilation, string or undefined is not assignable to string index type string.

[ghost]

There were the following issues with this Pull Request

• Commit: 8eec281
  • ✖ message may not be empty
  • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.

---

🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

[tinesoft]

FYI, you could have simply reopened PR #31 and edited both description and commit, to avoid duplicating the issue...

Ok, now at least we have a clear context, and the changes you made are more coherent with the issue. 😄
To take it even further, you can even completely delete the line:

 [key: string]: 'deny' | 'allow' | 'dismiss';

(i dont recall any valid reason why it was there in the first place...)

Once done, please update you commit message to follow the Angular git:

fix(typings): fix lint isssue due to bad typings in NgcCookieConsentStatus interface

Please git --amend and git push --force the commit instead of making a new one

Thanks in advance

[ghost]

There were the following issues with this Pull Request

• Commit: 8eec281
  • ✖ message may not be empty
  • ✖ type may not be empty
• Commit: d7adfe4
  • ✖ message may not be empty
  • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.

---

🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

[sylvainonweb]

So, is there a chance this pull request to be merged ?